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.

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

Using some wildcards in Curl accept header causes 404 #159

Closed
drn05r opened this issue Jul 26, 2021 · 3 comments
Closed

Using some wildcards in Curl accept header causes 404 #159

drn05r opened this issue Jul 26, 2021 · 3 comments
Assignees
Labels
Milestone

Comments

@drn05r
Copy link
Collaborator

@drn05r drn05r commented Jul 26, 2021

If you make a request using Curl like:

curl -v -H 'Accept: text/,application/' https://example.eprints.org/id/eprint/123/

Then you get a 404 error rather than the expected page, the following Curl requests work fine and return a 200:

@drn05r drn05r added the bug label Jul 26, 2021
@drn05r drn05r added this to the 3.4.4 milestone Jul 26, 2021
@drn05r drn05r self-assigned this Jul 26, 2021
@drn05r
Copy link
Collaborator Author

@drn05r drn05r commented Jul 26, 2021

There has been a suggestion that the following line maybe at issue:

perl_lib/EPrints/Apache/CRUD.pm [line 953] - elsif( $subtype eq '*' ) {}

@drn05r
Copy link
Collaborator Author

@drn05r drn05r commented Jul 26, 2021

This does not look like it hits CRUD.pm at least not line 953. It may be in Apache/Auth.pm or Apache/Rewrite.pm.

@drn05r
Copy link
Collaborator Author

@drn05r drn05r commented Jul 27, 2021

Fixed by commit c6c862b

@drn05r drn05r closed this Jul 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant