Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: Remove auth check for cloud tasks #171

Merged
merged 17 commits into from
Feb 13, 2024
Merged

fix: Remove auth check for cloud tasks #171

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
2952365
fix: Remove auth check for cloud tasks
exaby73 Feb 2, 2024
1b7fe53
skip app token check as well
exaby73 Feb 2, 2024
730dcb9
Add debug logging for auth_token and app_token
exaby73 Feb 2, 2024
3b7f1a0
Update logger statements in util.py
exaby73 Feb 2, 2024
be79553
Refactor logger import in util.py
exaby73 Feb 2, 2024
6baa839
Add debug logs for Authorization and X-Firebase-AppCheck headers
exaby73 Feb 2, 2024
999f0c9
Refactor debug log statements in util.py
exaby73 Feb 2, 2024
2fed3a6
Refactor token verification logic in util.py and tasks_fn.py
exaby73 Feb 2, 2024
a989391
Revert on_task_dispatched function to include enforce_app_check param…
exaby73 Feb 2, 2024
a7da435
Fix error logging in _on_call_handler function
exaby73 Feb 2, 2024
24b777d
Add debug logging for auth token
exaby73 Feb 2, 2024
3ef480e
Add check for uid in auth_token
exaby73 Feb 3, 2024
ec14738
Remove unused logger import and log statement
exaby73 Feb 3, 2024
977c517
Lint fixes
exaby73 Feb 3, 2024
588a60d
Formatting fix
exaby73 Feb 3, 2024
0cacaa9
Update AuthData class to allow None for uid
exaby73 Feb 7, 2024
390d2a8
Add check for None value in auth variable
exaby73 Feb 8, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add check for uid in auth_token
  • Loading branch information
@exaby73
exaby73 committed Feb 3, 2024
commit 3ef480e10afd8543411e5ec710c3e383758cb8a4
3 changes: 1 addition & 2 deletions src/firebase_functions/https_fn.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -377,8 +377,7 @@ def _on_call_handler(func: _C2,
token_status.app_token),
)

if token_status.auth_token is not None:
_logging.debug(token_status.auth_token)
if token_status.auth_token is not None and "uid" in token_status.auth_token:
context = _dataclasses.replace(
context,
auth=AuthData(token_status.auth_token["uid"],
Expand Down
Loading