[16.0][FIX] website_require_login: Error response#1099
Open
Alexgars73 wants to merge 1 commit intoOCA:16.0from
Open
[16.0][FIX] website_require_login: Error response#1099Alexgars73 wants to merge 1 commit intoOCA:16.0from
Alexgars73 wants to merge 1 commit intoOCA:16.0from
Conversation
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
Contributor
Author
|
@pedrobaeza could you remove the stale tag please? |
pedrobaeza
removed
the
stale
Member
|
OK, but please enroll people to review it. |
Contributor
Author
|
@yvaucher @NICO-SOLUTIONS could you review please? |
NICO-SOLUTIONS
approved these changes
Aug 1, 2025
Alexgars73
force-pushed
the
16.0-fix-website_require_login
branch
2 times, most recently
from
c5c4a3e to
58037bf
Compare
tarteo
requested changes
Oct 10, 2025
| ) | ||
|
|
||
| def test_dispatch_json_no_redirect(self): | ||
| """Con Content-Type application/json no debe haber redirección.""" |
Alexgars73
changed the title
Alexgars73
force-pushed
the
16.0-fix-website_require_login
branch
from
58037bf to
fb35027
Compare
SirPyTech
approved these changes
Jan 21, 2026
| path = request.httprequest.path | ||
| for auth_path in auth_paths: | ||
| if auth_path == path or Path(auth_path) in Path(path).parents: | ||
| if request.httprequest.content_type == "application/json": |
There was a problem hiding this comment.
suggestion: I'd place this close to the checks at the beginning of the method so that if this fails then we can skip everything else, like the search method.
| "Expected the response status code to be 200 which means no redirection", | ||
| ) | ||
|
|
||
| def test_dispatch_json_no_redirect(self): |
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
6 participants
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.
Error in response in first load:
