-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bugfix: multisite token logic #51
Draft
defunctl
wants to merge
117
commits into
main
Choose a base branch
from
bugfix/multisite-token-logic
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 3 commits
Commits
Show all changes
117 commits
Select commit
Hold shift + click to select a range
639f785
Fix HTTP mocks not working after base url was renamed.
defunctl 98a4bfb
Clean up test
defunctl 550dc15
Add ability to mock a plugin is activated
defunctl c59a64c
WIP: refactor token auth for multisite using plugin
defunctl ce82145
Include slug in token disconnect logic
defunctl 3346f9c
Refactor Authorized
defunctl a65371d
WIP: conditional token storage
defunctl d14edea
WIP: multisite logic refactor
defunctl 46f3eb4
Add get_license_key function
defunctl 521a8a0
Reset multisite config to original state after each test
defunctl 0270487
Refactor get_domain() to take into consideration unique subsites when…
defunctl 1e2f595
Remove token check, replace with main site check
defunctl 4438ee2
Add Config::allows_network_licenses() and update Multisite_Main_Site.php
defunctl 16ef300
Add a hash suffix to the main site URL when on multisite with network…
defunctl 8d27445
Delete old pipeline steps
defunctl a9b95be
Add more helper functions
defunctl 86325a0
Rename get_domain function
defunctl 5a267e8
Move disconnect URL into its own method, add helper function
defunctl f1fad44
Allow getting the original domain name
defunctl 92f883c
Get original domain for download args
defunctl a12acd1
Remove get_token, update get_authorization_token
defunctl 7840ed2
Delete unneeded Authorized class
defunctl bb1b85d
Fix license manager caching
defunctl d558d26
Clean up functions, add is_user_authorized()
defunctl 77e0e48
Add and adjust functions, first pass at multisite licensing documenta…
defunctl 8e233f1
Rollback is_authorized() change, add a new function instead
defunctl 5a5aee0
Fix warning
defunctl 0f7e720
Refactor allows_multisite_license() to accept an already made resource
defunctl 7dc8c71
Fix comment
defunctl 2cef62a
Add set_license_key example
defunctl f334ac1
Fix set_license_key function
defunctl f4b9d9a
Remove stray ampersand
defunctl 075bbe0
formatting
defunctl ba8721b
Attempt to allow network validation
defunctl eb95d51
Show the correct message based on network validation
defunctl 158dff9
submit to network settings if on the network_admin
defunctl d6cbf16
Add is_license_valid() function
defunctl a0a528e
rename to validate_license, return entire response
defunctl aafab54
Don't return null on empty license, validate allows a null key
defunctl d204a79
Remove unused container property, add types
defunctl fe207d0
Add nullable type
defunctl a2e9b87
Use dependency injection for Ajax
defunctl 019e264
ensure we are multisite aware when fetching the license key for the s…
defunctl b028d36
Fix bad return type
defunctl 200afd4
Show default licensing messages if coming from the network admin
defunctl 10abc3a
Store key status in network if required
defunctl c5d2d58
Update the license key valid cache after updating key locally
defunctl 1b34a0a
Ensure key is network at network/local level when validating
defunctl 9a161d7
Don't register settings during ajax requests
defunctl 3ee1c73
Ensure to validate license key before storing when callback connecting
defunctl 835c01e
Ensure key being checked is network aware during license validation
defunctl ae65a26
set_license_key() will properly update the validation state of the li…
defunctl e07e4b0
Merge branch 'feature/authorization-caching' into bugfix/multisite-to…
defunctl 6093e45
Update base testcase to fix merge issue and use config::reset()
defunctl 32a67b1
Merge branch 'fix/ESM-70-fatal-error-on-ticket-transfer' into bugfix/…
defunctl 95006d7
Merge branch 'main' into bugfix/multisite-token-logic
defunctl deeda32
Merge branch 'feat/enable-auto-updates' into bugfix/multisite-token-l…
kadencewp 6e17901
Use unique actions for each resource slug for token connect/disconnect
defunctl 282fca3
Remove prefix from token connect/disconnect actions, we don't have th…
defunctl 062fa1f
Bugfix: Ensure download_url property is not null before attempting to…
defunctl 373f6a5
Add action to disconnect controller
defunctl 4d8d99c
Properly fix Undefined property: stdClass::$download_url
defunctl 6bd55c1
Merge branch 'bugfix/multisite-token-logic' into bugfix/token-connect…
defunctl 93426a9
Refactor to use uplink_slug and no longer use WordPress's action quer…
defunctl 20616d7
Formatting
defunctl 3f2a0e6
Use a constant for the auth action
defunctl f77ede3
Add ActionManagerTest.php
defunctl fb9c439
Move assertion into single loop
defunctl da935ee
Use standard action format with prefix
defunctl 3e5df6b
Update connect/disconnect hook name
defunctl 00e750a
Fix action docblocks
defunctl 15a41bc
call correct action in tests
defunctl a1fd036
Formatting
defunctl b70ad24
Fix spelling
defunctl 95f44c9
Formatting
defunctl fd7a2c0
Merge pull request #59 from stellarwp/bugfix/token-connect-disconnect…
defunctl 4557d04
Merge remote-tracking branch 'origin/bugfix/prevent-empty-class' into…
defunctl bf9880a
Merge remote-tracking branch 'origin/feat/allow-disabling-updates' in…
defunctl cf87f7c
Make authorized checks require a plugin slug
defunctl 24445ff
Fix missing variable, update README.md
defunctl f24a385
Don't use slug for transient cache key
defunctl a6ceefa
Only use the token to make the cache key
defunctl bfebca6
Refactor token disconnect logic to also delete the authorization cache
defunctl 138a834
Properly build a cache hash with just a token, don't use prefix in qu…
defunctl 1c7e27d
Merge pull request #64 from stellarwp/feature/ms/authorized-check-to-…
defunctl f367e86
Merge branch 'main' into bugfix/multisite-token-logic
defunctl 66d9b5e
Show WP info
defunctl c5f5461
use the correct env var to call slic
defunctl ec95ca3
Fix tests failing due to Unexpected incorrect usage notice for wp_add…
defunctl 6a1ecc4
Bump actions/checkout versions, fix composer cache dir
defunctl 213a1d7
Manually bring in updated tests matrix
defunctl d3374a8
Fix type error in filter_upgrader_pre_download
defunctl 19dcdfa
Merge branch 'main' into bugfix/KAD-2308/updater-type-error
defunctl e84dc60
Merge branch 'bugfix/KAD-2308/updater-type-error' into bugfix/multisi…
defunctl 1f8c277
Fix typo
defunctl 6aa9ebe
Merge branch 'bugfix/KAD-2308/updater-type-error' into bugfix/multisi…
defunctl 0d69fe0
Force type for delete_transient kadence zendesk ticket 610295
defunctl 775b54f
Force set_transient type for good measure
defunctl d8b9855
Allow some HTML tags in notices, show error if nonce failed and lites…
defunctl d37e2bb
Fix allowed tags
defunctl 81e6cb1
add filter
kadencewp 0b471da
fix test
kadencewp 370bd50
updates with more escaping and not showing a button unless it's TEC o…
kadencewp 0e03db7
test update
kadencewp 5c6e94d
Use require_once as a statement
defunctl 64eda1e
Remove deprecated config, ignore require_once errors
defunctl 8458885
Bump action versions
defunctl ac42ee4
Merge pull request #90 from stellarwp/chore/phpstan-updates
defunctl b7553a7
Merge branch 'bugfix/multisite-token-logic' into bugfix/missing-filter
defunctl 0f8f4b8
Fix array
kadencewp 007ca4b
Add test
kadencewp 6983bea
fix string
kadencewp bb787f2
target..
kadencewp 301a5d4
Merge pull request #89 from stellarwp/bugfix/missing-filter
kadencewp d134ede
Account for an empty siteurl in the database
kadencewp 82d3e19
apply change
kadencewp f33e2d3
Merge pull request #93 from stellarwp/bugfix/empty-domain
kadencewp File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a line break!