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

New .NET-backend #1210

Closed
sweco-semtsg opened this issue Oct 11, 2022 · 1 comment
Closed

New .NET-backend #1210

sweco-semtsg opened this issue Oct 11, 2022 · 1 comment
Assignees
@sweco-semtsg @sweco-semtto
Labels
module:backend MapService stuff .NET Pull requests that update .net code new feature Request for adding/changing functionality
Milestone
3.13

Comments

@sweco-semtsg
Copy link
Contributor

Update/redevelop .NET-backend to .NET 6 according to the attached OpenAPI-specification.
@jacobwod jacobwod added this to the 3.x milestone Oct 12, 2022
@jacobwod jacobwod added module:backend MapService stuff new feature Request for adding/changing functionality .NET Pull requests that update .net code and removed idea labels Nov 17, 2022
@jacobwod jacobwod pinned this issue Dec 15, 2022
jacobwod added a commit that referenced this issue Dec 15, 2022
@jacobwod
Changed response on /informative/load/:name to 404:
f3892db 
- As part of #1210 it was decided that 404 should be sent when a document can't be loaded. We should also simplify the error message, not to show the entire path to file.
- The reality is that the we now send 404, even if the real error can have another cause (e.g. not sufficient permissions to read file - i.e. not allowed, rather than not found).
- To simplify things we now send 404 no matter why. To allow easy debugging, we log the actual error message in the log. This way admins can identify and fix problems.
- Also a small fix for Admin UI: don't try to proceed with parsing if response is not 200. This was an error before and resulted in a crash (no matter if we sent 500 or 404), but it's nice to fix nonetheless.
@sweco-semtto sweco-semtto mentioned this issue Mar 3, 2023
Closed
@jacobwod jacobwod mentioned this issue Mar 29, 2023
Closed
@jacobwod jacobwod unpinned this issue Sep 22, 2023
@jacobwod jacobwod modified the milestones: 3.x, 3.13 Mar 7, 2024
@jacobwod
Copy link
Member

jacobwod commented Mar 7, 2024

Merged in #1395

@github-project-automation github-project-automation bot moved this to Done in Hajk Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sweco-semtsg sweco-semtsg

@sweco-semtto sweco-semtto

Labels
module:backend MapService stuff .NET Pull requests that update .net code new feature Request for adding/changing functionality
Projects
Hajk
Status: Done
Milestone
3.13
Development

No branches or pull requests
3 participants
@jacobwod @sweco-semtsg @sweco-semtto