Normalize to the Prolific plugin standard (1.1.0) - #2
Open
millertchris wants to merge 7 commits into
Open
millertchris wants to merge 7 commits into
millertchris wants to merge 7 commits into
Conversation
… license server Adds composer.json/composer.lock with yahnis-elsts/plugin-update-checker ^5.7 and includes/update-checker.php, which checks api.prolificdigital.io using the PROLIFIC_LICENSE_KEY_BETTERSTACK_LOGGER constant or the prolific_license_key filter. vendor/ is gitignored. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Standard header (WordPress 6.5+, PHP 8.4+, GPL-2.0-or-later, Update URI) with a BETTERSTACK_LOGGER_VERSION constant. The WPINC guards become the one-line ABSPATH guard, and translations load from languages/. The Short Description and Tags header lines are dropped (tags live in readme.txt). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Deletes betterstack_api_key, betterstack_error_logging_enabled and betterstack_event_logging_enabled. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…dme.txt for 1.1.0 readme.txt is rewritten in WordPress.org format, and README.md gains updates, stored-data, development and release sections. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…oken create_log_entry() called wp_remote_post() with sslverify => false, so log messages and the Bearer source token could be intercepted by anyone on the network path. The override is removed and WordPress's default certificate verification applies. It was the plugin's only wp_remote_* request. register_setting() now sanitizes betterstack_api_key (Bearer token characters only, via sanitize_text_field) and stores the two logging checkboxes only as "yes" or "". The token was already escaped with esc_attr() on the settings screen. Adds tests/security/tls-and-token.php (wp eval-file), which captures the request args through http_request_args with a mocked pre_http_request, and checks sanitizing and escaping. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Normalizes BetterStack Logger to the Prolific plugin standard (slug
betterstack-logger, version 1.1.0).What changes
betterstack-logger, Update URI on the license server.api.prolificdigital.io; bundled update-checker copies and GitHub-token code removed..distignore/ phpcs config.prolific-digital/.github.fix(security)commits and the CHANGELOG Security section.Before merging: repo renames and the license-server registry deploy go out together, and releases are tagged after merge. See CHANGELOG.md for upgrade notes (e.g. re-activation after a main-file rename).
Commits
🤖 Generated with Claude Code