Skip to content

CHEF-23586: Update license handling for SaaS users - #1920

Merged
mwrock merged 2 commits into
mainfrom
agam/CHEF-23586_1
Jul 23, 2025
Merged

CHEF-23586: Update license handling for SaaS users#1920
mwrock merged 2 commits into
mainfrom
agam/CHEF-23586_1

Conversation

@agmathur

@agmathur agmathur commented Jul 18, 2025

Copy link
Copy Markdown
Collaborator

Description:

  1. On-prem (non SaaS) users may not be signed or have valid license to access any routes.
  2. SaaS users with valid license are allowed to access all routes after signing in and are by default moved to /origins page.
  3. SaaS users without license get 404 code and after signed-in has to stay on /profile page till license is provided.
  4. SaaS users with expired license should also stay on /profile page after sign-in till valid license is provided.
  5. On refresh the users should remain on same page, if still signed-in.

Issues:

https://progresssoftware.atlassian.net/browse/CHEF-23586

Recordings:

On-prem (hosted=false)

On-prem.mp4

SaaS (hosted=true)

SAS.mp4

agmathur added 2 commits July 18, 2025 11:42
Signed-off-by: Agam Mathur <agmathur@progress.com>
Signed-off-by: Agam Mathur <agmathur@progress.com>
@sonarqube-for-infrastructure-prod

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@vkarve-chef vkarve-chef changed the title CHEF-23586: Update license handling for SAS users CHEF-23586: Update license handling for SaaS users Jul 18, 2025
@mwrock
mwrock merged commit 42b3132 into main Jul 23, 2025
29 of 30 checks passed
@mwrock
mwrock deleted the agam/CHEF-23586_1 branch July 23, 2025 13:33
chef-expeditor Bot pushed a commit that referenced this pull request Jul 23, 2025
Obvious fix; these changes are the result of automation not creative thinking.
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