Skip to content
This repository was archived by the owner on Dec 12, 2018. It is now read-only.

Conversation

@Stephan202
Copy link

Rationale: this is a client error for which no server-side action is required.
Additionally, elsewhere in the class a similar error is also logged at debug
level.

This fixes #1076

@Stephan202
Copy link
Author

(The Travis CI failure seems completely unrelated to my change...)

Rationale: this is a client error for which no server-side action is required.
Additionally, elsewhere in the class a similar error is also logged at debug
level.

This fixes #1076
@Stephan202 Stephan202 changed the base branch from master to okta September 19, 2017 05:47
@Stephan202
Copy link
Author

Updated the PR to target the okta branch. Is the Stormpath team interested in merging this change?

@Stephan202
Copy link
Author

Development of this code base seems to have fully ceased. Will leave the patch here in a comment and drop the associated fork.

Date:   Wed Mar 8 18:39:39 2017 +0100

    Log invalid JWT token exception at debug level
    
    Rationale: this is a client error for which no server-side action is required.
    Additionally, elsewhere in the class a similar error is also logged at debug
    level.
    
    This fixes #1076

diff --git a/extensions/servlet/src/main/java/com/stormpath/sdk/servlet/filter/account/CookieAccountResolver.java b/extensions/servlet/src/main/java/com/stormpath/sdk/servlet/filter/account/CookieAccountResolver.java
index 12bbf69..12b865d 100644
--- a/extensions/servlet/src/main/java/com/stormpath/sdk/servlet/filter/account/CookieAccountResolver.java
+++ b/extensions/servlet/src/main/java/com/stormpath/sdk/servlet/filter/account/CookieAccountResolver.java
@@ -141,7 +141,7 @@ public class CookieAccountResolver implements Resolver<Account> {
 
         } catch (Exception e) {
             String msg = "Encountered invalid JWT in refresh_token cookie. We will now delete both the access and refresh cookies for safety.";
-            log.error(msg, e);
+            log.debug(msg, e);
             deleteCookie(response, cookie);
             deleteCookie(response, accessTokenCookieResolver.get(request, response));
         }
@Stephan202 Stephan202 closed this Nov 11, 2018
@Stephan202 Stephan202 deleted the bugfix/issue-1076 branch November 11, 2018 19:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

1 participant