Repository navigation
BCLOUD-14839: Add client attestation support. SetAppCheckToken and Se… - #397
Open
pierrepbitheads wants to merge 4 commits into
Open
pierrepbitheads wants to merge 4 commits into
pierrepbitheads wants to merge 4 commits into
Conversation
…tAppCheckTokenProvider methods.
michaelc-bitheads
left a comment
michaelc-bitheads
left a comment
Contributor
There was a problem hiding this comment.
I have several comments... also C.C. @stevemrjones to see if he agrees with me about some of my stronger opinions, such as the README and TestAppCheck.
Contributor
|
Also are there any logs about this? What does the success and failure callback look like? That should be posted in the main body of the PR. |
… token provider timeout
…ts related section.
Contributor
Author
|
Added output logs to the PR description for the various tests. |
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
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.
BCLOUD-14839: Added client attestation support to csharp library. SetAppCheckToken and SetAppCheckTokenProvider methods.
Runtime outputs for the various tests:
Valid - Token provider
Apple App Attest - device
SDK provider will request a fresh Firebase token before authentication.
Success log:
Status: 200 Reason: 0
Valid token accepted. Repeat Valid to verify saved identity.
Failure log:
Status: 900 Reason: 90001
Timeout trying to reach brainCloud server
Omitted - Token provider
Apple App Attest - device
Firebase bypassed; provider removed. Testing backend enforcement.
Success log:
Status: 403 Reason: 40300
Processing exception (message): App Check Token is required
Failure log:
Status: 900 Reason: 90001
Timeout trying to reach brainCloud server
Invalid - Token provider
Apple App Attest - device
Firebase bypassed; provider removed. Testing backend enforcement.
Success log:
Status: 403 Reason: 40300
Processing exception (message): Invalid App Check Token
Failure log:
Status: 900 Reason: 90001
Timeout trying to reach brainCloud server
Valid - Token setter
Apple App Attest - device
Fetching a fresh Firebase token for SetAppCheckToken...
Success log:
Status: 200 Reason: 0
Valid token accepted. Repeat Valid to verify saved identity.
Failure log:
Status: 900 Reason: 90001
Timeout trying to reach brainCloud server
Omitted - Token setter
Apple App Attest - device
Firebase bypassed; provider removed. Testing backend enforcement.
Success log:
Status: 403 Reason: 40300
Processing exception (message): App Check Token is required
Failure log:
Status: 900 Reason: 90001
Timeout trying to reach brainCloud server
Invalid - Token setter
Apple App Attest - device
Firebase bypassed; provider removed. Testing backend enforcement.
Success log:
Status: 403 Reason: 40300
Processing exception (message): Invalid App Check Token
Failure log:
Status: 900 Reason: 90001
Timeout trying to reach brainCloud server