Bugfix/no push reregister on token refresh#1003
Open
franco-zalamena-iterable wants to merge 10 commits intoautoretry-masterfrom
Open
Bugfix/no push reregister on token refresh#1003franco-zalamena-iterable wants to merge 10 commits intoautoretry-masterfrom
franco-zalamena-iterable wants to merge 10 commits intoautoretry-masterfrom
Conversation
Includes 1. getRemoteConfiguration checking for the flag. And assuming false if no values exist. Also stores in sharedPreferences to load from it next time. 2. isAutoRetryOnJWTFailure on `IterableAPI` which other classes with use. Not public. 3. Introduced three states which Taskmanager can rely on. VALID, INVALID, UNKNOWN. 4. Listener pub/sub added which will invoke onAuthTokenReady() on listner class. Adding array of AuthTokenReadyListener on Authmanager and its add and remove list methods. 5. Right now authHandler null returns true hoping to bypass jwt. But JWT based request will fail if API needs jwt. This could pile up storage. But this seems like a valid scenario 6. handleAuthTokenSucess sets authtoken to unknown when a new token is set. I hope it should be right approach. Because its new and unknown at this point. 7. `IterableConfig` has final boolean `autoRetryOnJwtFailure`. Its not something I want customers to be able to configure. It should be removed before going to master. It should be internal only variable. 8. IterableRequestTask - has some changes introduced due to the flow not falling under jwt failure. Response code coming has -1 was the root cause. Hence the change around responseCode >= 0 && responseCode < 400 9. `IterableRequestTask` - Line 215 - Still not sure if RequestProcessor check should happen. It does make sense to have it as only offline stored request will be reprocessed. And this feature should only work for those requests going through offline Request processor 10. `IterableRequestTask` - fromJSON method change is something I havent checked properly. I think its for unit testing the changes are done. 11. IterableTaskRunner. Would like to see if I can remove the method - isJWTFailure(responseCode). May be its a small abstraction thats needed. 12. Havent checked unit tests properly yet
franco-zalamena-iterable
force-pushed
the
bugfix/no-push-reregister-on-token-refresh
branch
from
68f10bc to
6bc8f7b
Compare
Aligns offline task retry logic with iOS PR 1012: use HTTP status codes instead of fragile string matching to classify errors. 4xx (except 401 JWT and 429 rate-limit) are permanent failures (task deleted). 5xx, network errors, and timeouts are transient (task retained for retry). Also fixes two tests that were missing mock server responses, which now correctly matter since network errors are treated as transient. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Now we are recreating the token and starting the auto retry
franco-zalamena-iterable
force-pushed
the
bugfix/no-push-reregister-on-token-refresh
branch
from
db3f49f to
bc065e5
Compare
Contributor
Author
|
Wait for bugbash to be completed |
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.
✏️ Description
Fixed a problem with completeuserlogin retrigger
Decoupling the completeUserLogin from the SetAuthToken flow. Those live independently now