Skip to content

fix(feeds): Test errors after save#2371

Merged
zze0s merged 1 commit intodevelopfrom
fix/feeds-test-broken-after-save
Mar 8, 2026
Merged

fix(feeds): Test errors after save#2371
zze0s merged 1 commit intodevelopfrom
fix/feeds-test-broken-after-save

Conversation

@zze0s
Copy link
Copy Markdown
Collaborator

@zze0s zze0s commented Mar 8, 2026

What is the relevant ticket/issue

  • Reported on Discord

What's this PR do?

After a Feed has been saved and a user clicks Test it uses the literal <redacted> value as API key.

Add
  • Check for existing ID, then check if it's a redacted value, if yes set api key and cookie

Where should the reviewer start?

  • First file

How should this be manually tested?

  • Create a Feed, click Test it should work, then open with edit and click test button again and it should work

Risk involved?

  • Low

Does the documentation or dependencies need an update?

  • No

@zze0s zze0s added this to the v1.74.0 milestone Mar 8, 2026
@zze0s zze0s added bug Something isn't working backend Backend feeds Feeds: RSS, Torznab and Newznab labels Mar 8, 2026
@zze0s zze0s moved this to Review in Autobrr development Mar 8, 2026
@zze0s zze0s changed the title fix(feeds): Test broken after save fix(feeds): Test errors after save Mar 8, 2026
@zze0s zze0s merged commit e0ee4ce into develop Mar 8, 2026
35 checks passed
@github-project-automation github-project-automation bot moved this from Review to Done in Autobrr development Mar 8, 2026
@zze0s zze0s deleted the fix/feeds-test-broken-after-save branch March 8, 2026 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Backend bug Something isn't working feeds Feeds: RSS, Torznab and Newznab

Projects

Development

Successfully merging this pull request may close these issues.

1 participant