Skip to content
This repository was archived by the owner on Jan 23, 2025. It is now read-only.

add refresh token logic - direct challange service #310

Merged
merged 2 commits into from
Aug 23, 2017

Conversation

deedee
Copy link
Contributor

@deedee deedee commented Aug 23, 2017

No description provided.

@ajefts ajefts merged commit 52e00a7 into topcoder-archive:dev Aug 23, 2017
@ajefts
Copy link
Contributor

ajefts commented Aug 23, 2017

fyi @skyhit @deedee I merged this in but the build is failing. Could you take a look? I think Jenkins emailed you the log...

@deedee
Copy link
Contributor Author

deedee commented Aug 23, 2017

@ajefts Could send to me the log files?

@ajefts
Copy link
Contributor

ajefts commented Aug 24, 2017

These are the errors

dev/src/java/main/com/topcoder/direct/services/view/action/ServiceBackendDataTablesAction.java:36: cannot find symbol
    [javac] symbol  : class StandardCharsets
    [javac] location: package java.nio.charset
    [javac] import java.nio.charset.StandardCharsets;
    [javac]                        ^

dev/src/java/main/com/topcoder/direct/services/view/action/ServiceBackendDataTablesAction.java:823: cannot find symbol
    [javac] symbol  : variable StandardCharsets
    [javac] location: class com.topcoder.direct.services.view.action.ServiceBackendDataTablesAction
    [javac]                 String payload = new String(Base64.decodeBase64(payloadStr.toString().getBytes(StandardCharsets.UTF_8)));
    [javac]  

}

validateCookieV2V3(jwtCookieV2,jwtCookieV3);
Copy link
Collaborator

Choose a reason for hiding this comment

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

@deedee should we get the cookies again?

Copy link
Contributor Author

@deedee deedee Aug 24, 2017

Choose a reason for hiding this comment

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

Do you mean why it check twice?I think we need to check there since that method called by some action directly i.e /my/getMyCreatedChallenges. If it is valid will not be refreshed

@deedee
Copy link
Contributor Author

deedee commented Aug 24, 2017

@ajefts you need to use jdk 7

@deedee
Copy link
Contributor Author

deedee commented Aug 24, 2017

@ajefts this should work with jdk6/7 #311

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants