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

Fix nullable and error handling in AASXFileServerAPIApiController #339

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Freezor
Copy link
Contributor

@Freezor Freezor commented Jul 31, 2024

Description

Fixed the nullable handling to represent the logic that was present before the .Net8.0 change. Also made sure that no new exceptions are thrown based on the nullable changes

@Freezor Freezor added the bug Something isn't working label Jul 31, 2024
@juileetikekar
Copy link
Contributor

Hi @Freezor,

I tried to test the PR. All the APIs in File Server IF work fine, except for Get /Packages. It returns 406 error code with the warning message in the logs: "No output formatter was found for content types '' to write the response.". Ideally, it should return paginated result with a list of packageIds & aasIds, i.e., list of "PackageDescription". The functionaloty was working as expected earlier, so something must have changed during refactoring.

image

I would really appreciate, if you could look into it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants