Skip to content

Commit 19834be

Browse files
committed
Merge tag '23.2.1' into develop
Fix SSO error handling for inactive users
2 parents f5dc03e + 32bb4fa commit 19834be

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,11 @@
22

33
We follow the CalVer (https://calver.org/) versioning scheme: YY.MINOR.MICRO.
44

5+
23.2.1 (07-07-2023)
6+
===================
7+
8+
* Fix SSO error handling for inactive users
9+
510
23.2.0 (06-25-2023)
611
===================
712

src/main/java/io/cos/cas/osf/web/flow/login/OsfPrincipalFromNonInteractiveCredentialsAction.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
import com.google.gson.JsonParser;
88

99
import io.cos.cas.osf.authentication.credential.OsfPostgresCredential;
10+
import io.cos.cas.osf.authentication.exception.InstitutionSsoAccountInactiveException;
1011
import io.cos.cas.osf.authentication.exception.InstitutionSsoAttributeMissingException;
1112
import io.cos.cas.osf.authentication.exception.InstitutionSsoAttributeParsingException;
1213
import io.cos.cas.osf.authentication.exception.InstitutionSsoDuplicateIdentityException;
@@ -788,7 +789,7 @@ private OsfApiInstitutionAuthenticationResult notifyOsfApiOfInstnAuthnSuccess(
788789
}
789790
if (OsfApiPermissionDenied.INSTITUTION_SSO_ACCOUNT_INACTIVE.getId().equals(errorDetail)) {
790791
LOGGER.error("[OSF API] Failure - Inactive Account: {}", ssoUser);
791-
throw new InstitutionSsoDuplicateIdentityException("OSF API denies inactive account");
792+
throw new InstitutionSsoAccountInactiveException("OSF API denies inactive account");
792793
}
793794
}
794795
// Handle unidentified HTTP 403 FORBIDDEN failures

0 commit comments

Comments
 (0)