Skip to content

[ACS-10385] Fix missing "WWW-Authenticate" header 9.N backport#73

Merged
cezary-witkowski merged 1 commit into
support/SP/9.Nfrom
fix/ACS-10385-rss-subscription-9-n-backport
Oct 1, 2025
Merged

[ACS-10385] Fix missing "WWW-Authenticate" header 9.N backport#73
cezary-witkowski merged 1 commit into
support/SP/9.Nfrom
fix/ACS-10385-rss-subscription-9-n-backport

Conversation

@cezary-witkowski

@cezary-witkowski cezary-witkowski commented Oct 1, 2025

Copy link
Copy Markdown

Backporting #72

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a missing "WWW-Authenticate" header issue by correcting the order of operations in an authentication error response and updates GitHub Actions cache dependencies to a more recent version.

  • Moves the "WWW-Authenticate" header setting before the error response is sent to ensure proper HTTP authentication flow
  • Updates GitHub Actions cache action from v1 to v4 across multiple workflow jobs

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
FeedController.java Reorders header setting to occur before sending the HTTP error response
build.yml Updates cache action version from v1 to v4 in three workflow jobs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@cezary-witkowski
cezary-witkowski merged commit d0828d8 into support/SP/9.N Oct 1, 2025
3 checks passed
@cezary-witkowski
cezary-witkowski deleted the fix/ACS-10385-rss-subscription-9-n-backport branch October 1, 2025 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants