Skip to content

fix(auth): more logging for investigation#80867

Merged
sentaur-athena merged 1 commit into
masterfrom
athena/more-logging
Nov 16, 2024
Merged

fix(auth): more logging for investigation#80867
sentaur-athena merged 1 commit into
masterfrom
athena/more-logging

Conversation

@sentaur-athena

Copy link
Copy Markdown
Member

I see in the logs that this logs shows up although the target_org_id_or_slug is None. I want to see what kind of urls cause this issue so adding more to the log.

@sentaur-athena sentaur-athena requested a review from a team as a code owner November 15, 2024 22:38
@sentaur-athena sentaur-athena requested review from a team November 15, 2024 22:38
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 15, 2024
# raise AuthenticationFailed("Unauthorized organization access")
logger.info(
"Token has access to organization %s but wants to get access to organization %s",
"Token has access to organization %s but wants to get access to organization %s: %s",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would it be easier to find this log if the %s fields were logged in an extra dict?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't matter much because I'm deleting it on Monday.

@sentaur-athena sentaur-athena merged commit b826d61 into master Nov 16, 2024
@sentaur-athena sentaur-athena deleted the athena/more-logging branch November 16, 2024 00:44
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants