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

[Bug]: Query Parameter (IdShort and Cursor) cannot be empty #295

Open
BohnSeb opened this issue Jun 21, 2024 · 2 comments
Open

[Bug]: Query Parameter (IdShort and Cursor) cannot be empty #295

BohnSeb opened this issue Jun 21, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@BohnSeb
Copy link

BohnSeb commented Jun 21, 2024

Contact Details

No response

Describe the bug

When using the /shells endpoint to get all shells there is an error stating that the cursor and idShort query parameters are not set.
In my opinion, this behaviour is absolute nonsense, especially because the idShort query parameter cannot be empty and needs to be a perfect match to return any result at all.

This is the returned error body:
{ "errors": { "cursor": [ "The cursor field is required." ], "idShort": [ "The idShort field is required." ] }, "type": "https://tools.ietf.org/html/rfc9110#section-15.5.1", "title": "One or more validation errors occurred.", "status": 400, "traceId": "00-0792b1f9467fdff4ed7eb4c2f79152dd-340538d57374bead-00" }

To reproduce

Just use the /shells GET without query parameters

Screenshots

No response

Expected Behaviour

All Shells in the server should be returned.

Version

v0.3.1.40-aasV3-alpha-latest

Operating System

Windows 10

Relevant log output

No response

@BohnSeb BohnSeb added the bug Something isn't working label Jun 21, 2024
@Freezor
Copy link
Contributor

Freezor commented Jun 21, 2024

@BohnSeb thanks for the issue.
We currently have some issues we're trying to fix asap. I look into it shortly. There will be a new prerelease soon, that should improve this topic

Freezor added a commit that referenced this issue Jun 22, 2024
* update release note builde

* fix nullables and routing
@Freezor
Copy link
Contributor

Freezor commented Jun 22, 2024

@BohnSeb please check v0.3.1.44-aasV3-alpha-latest. THis should solve your issue. Please inform us, if you found other issues

@Freezor Freezor self-assigned this Jun 26, 2024
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

No branches or pull requests

2 participants