Skip to content

Conversation

@stoty
Copy link
Contributor

@stoty stoty commented Oct 21, 2025

No description provided.

/**
* Determines whether the given response represents an authentication challenge, without
* changing the {@link AuthExchange} state.
* Determines whether the response is 401/407 response depending to the challengeType
Copy link
Member

Choose a reason for hiding this comment

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

@stoty Can we avoid using explicit status codes here and use a more generic "challenge" term?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, I reverted to the original wording, but added a reference to challangeType.

@ok2c ok2c merged commit 426ad2c into apache:master Oct 21, 2025
10 checks passed
@ok2c
Copy link
Member

ok2c commented Oct 21, 2025

@stoty Cherry-picked to 5.5.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants