mirror of
https://github.com/Prowlarr/Prowlarr.git
synced 2026-03-02 22:57:22 -05:00
v1/applications/* endpoints not logging 400 #407
Labels
No labels
Area: API
Area: Database
Area: Db-migration
Area: Download Clients
Area: Indexer
Area: Metadata API
Area: Notifications
Area: Tooling
Area: UI
Area: Update API
Priority: High
Priority: Low
Priority: Medium
Status: Cannot Reproduce
Status: Confirmed
Status: Help Wanted
Status: In Progress
Status: Indexer - need invite
Status: Info Needed
Status: Investigating
Status: Logs Needed
Status: Maybe One Day
Status: Needs Triage
Status: Unlikely
Status: Waiting for OP
Status: Won't Fix
Type: Bug
Type: Bug
Type: Documentation
Type: Duplicate
Type: Enhancement
Type: External Bug
Type: Feature Request
Type: Regression
Type: Support
Type: Support.
lidarr-pull
radarr-pull
sonarr upstream
sonarr-pull
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/Prowlarr#407
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @rootwyrm on GitHub (Dec 11, 2021).
Is there an existing issue for this?
Current Behavior
Even at trace level, hits to the v1/applications/* endpoints which return error 400 are not logged anywhere. This is particularly obvious while troubleshooting 400 responses for partial-but-complete payloads.
Expected Behavior
API hits - particularly those resulting in errors - should be logged.
Steps To Reproduce
Slightly complicated. You'll need both a Sonarr and Prowlarr.
Extract the Prowlarr API key to /talecaster/shared/indexer.api and the Sonarr API key to /talecaster/shared/television.api
Must be done as:
Then you're gonna need this Python (in progress so it's messy.)
Note that a more complete payload in the browser is both logged and passes test fine. I'm trying to use minimum viable payload so I don't have to track categories (it's intended to roll with defaults there.)
Environment
What branch are you running?
Develop
Trace Logs?
There's about 10 or so tests in the middle of this. They just aren't logged. 10.1.1.50 is the browser client.
AB#2125
@rootwyrm commented on GitHub (Dec 11, 2021):
Again, the Python there is very in-progress, so lots of switching around. Forgot to mention that the behavior exhibits with both
v1/applications/testandv1/applications/Sonarrhere.@rootwyrm commented on GitHub (Dec 11, 2021):
Working browser payload:
Non-working API payload:
@Qstick commented on GitHub (Dec 11, 2021):
500 response on incomplete field set fixed here
github.com/Prowlarr/Prowlarr@62d15536dfPer discord
/applications/Sonarris not a valid endpoint, the application specifics will be set in the payload.@mynameisbogdan commented on GitHub (Apr 8, 2023):
Regarding the tags, it's related to #1080 and fixed by #1491.