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

Clear txt sid #108

Merged
merged 2 commits into from
Jul 4, 2021
Merged

Clear txt sid #108

merged 2 commits into from
Jul 4, 2021

Conversation

rohe
Copy link
Collaborator

@rohe rohe commented Jul 2, 2021 •
edited
Loading

Session IDs in the old type tokens was in clear text. Not so in the new ones.
This takes care of the difference.

rohe added 2 commits July 2, 2021 09:46
@rohe rohe requested review from nsklikas and peppelinux July 2, 2021 10:44
peppelinux
peppelinux approved these changes Jul 4, 2021
View reviewed changes
Copy link
Member

@peppelinux peppelinux left a comment

Choose a reason for hiding this comment

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

Code looks very good, I'll do my test this evening directly on the Dev branch, so let's merge this

@rohe
Copy link
Collaborator Author

rohe commented Jul 4, 2021

I'll wait to after lunch for @nsklikas to have his say.

@rohe rohe merged commit bef7365 into develop Jul 4, 2021
@rohe rohe deleted the clear_txt_sid branch July 4, 2021 12:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@nsklikas nsklikas Awaiting requested review from nsklikas

1 more reviewer

@peppelinux peppelinux peppelinux approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rohe @peppelinux