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

Server error with incorrect viewcode #3711

Open
PascalRepond opened this issue Jul 2, 2024 · 0 comments · May be fixed by #3780
Open

Server error with incorrect viewcode #3711

PascalRepond opened this issue Jul 2, 2024 · 0 comments · May be fixed by #3780
Assignees
@rerowep
Labels
bug Breaks something but is not blocking dev: sentry For PRs that are created to correct an error raised by Sentry. effort: minor Solved quickly

Comments

@PascalRepond
Copy link
Contributor

PascalRepond commented Jul 2, 2024 •
edited
Loading

Bug description:

If the public view URL contains an incorrect viewcode, some pages produce an internal error.

Examples:

Expected behavior:

An incorrect URL like this should produce a 404 not found error page.

Context

Sentry: https://sentryils.rero.ch:8443/organizations/sentry-rero-ils-prod/issues/122661/
@PascalRepond PascalRepond added the bug Breaks something but is not blocking label Jul 2, 2024
@PascalRepond PascalRepond added the dev: sentry For PRs that are created to correct an error raised by Sentry. label Jul 2, 2024
@PascalRepond PascalRepond moved this from Inbox to Product Backlog in RERO ILS issues Jul 2, 2024
@PascalRepond PascalRepond changed the title Server error with incorrect viecode Server error with incorrect viewcode Jul 2, 2024
@PascalRepond PascalRepond added the effort: minor Solved quickly label Aug 28, 2024
rerowep added a commit to rerowep/rero-ils that referenced this issue Nov 18, 2024
@rerowep
documents: fix incorrect viewcode
5d6c095 
* Closes: rero#3711

Co-Authored-by: Peter Weber <[email protected]>
@rerowep rerowep linked a pull request Nov 18, 2024 that will close this issue
Open
@rerowep rerowep self-assigned this Nov 18, 2024
rerowep added a commit to rerowep/rero-ils that referenced this issue Nov 19, 2024
@rerowep
documents: fix incorrect viewcode
92ca887 
* Closes: rero#3711

Co-Authored-by: Peter Weber <[email protected]>
rerowep added a commit to rerowep/rero-ils that referenced this issue Nov 20, 2024
@rerowep
documents: fix incorrect viewcode
dc8eeff 
* Closes: rero#3711

Co-Authored-by: Peter Weber <[email protected]>
@rerowep rerowep moved this from Approved to Ready to test in RERO ILS issues Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rerowep rerowep

Labels
bug Breaks something but is not blocking dev: sentry For PRs that are created to correct an error raised by Sentry. effort: minor Solved quickly
Projects
RERO ILS issues
Status: Ready to test
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

documents: fix incorrect viewcode rerowep/rero-ils
2 participants
@rerowep @PascalRepond