Skip to content

Commit

Permalink
Merge branch 'feature-PRESIDECMS-2965_fix-issues-with-numeric-rest-us…
Browse files Browse the repository at this point in the history
…er-ids' into release-10.28.0
  • Loading branch information
DominicWatson committed Oct 30, 2024
2 parents cffb07b + 00c072f commit 6ca636d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions system/services/rest/PresideRestAuthService.cfc
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ component {
var providerFeature = configuredProviders[ arguments.provider ].feature ?: "";
var coldbox = $getColdbox();

if ( providerFeature.len() && !$isFeatureEnabled( providerFeature ) ) {
if ( Len( providerFeature ) && !$isFeatureEnabled( providerFeature ) ) {
_failRequest( restRequest, restResponse, "Authentication provider [#provider#] is not enabled." );
return;
}
Expand All @@ -47,7 +47,7 @@ component {
, eventArguments = { restRequest=restRequest, restResponse=restResponse }
);

if ( IsSimpleValue( userId ?: {} ) && userId.len() ) {
if ( IsSimpleValue( userId ?: {} ) && Len( userId ) ) {
restRequest.setUser( userId );
return;
}
Expand Down Expand Up @@ -101,7 +101,7 @@ component {
, required any restResponse
, required string defaultStatusText
) {
var statusText = restResponse.getStatusText().len() ? restResponse.getStatusText() : arguments.defaultStatusText;
var statusText = Len( restResponse.getStatusText() ) ? restResponse.getStatusText() : arguments.defaultStatusText;

restRequest.finish();
if ( restResponse.getStatusCode() == 200 ) {
Expand Down

0 comments on commit 6ca636d

Please sign in to comment.