Skip to content
This repository has been archived by the owner on Mar 31, 2019. It is now read-only.

Add test for struct token validation issue #8

Merged
merged 2 commits into from Dec 3, 2018

Conversation

FossoresLP
Copy link
Owner

@FossoresLP FossoresLP commented Dec 1, 2018

As discussed in issue #7 trying to validate a token that has a struct as content can fail. This is caused by Go ordering the map alphabetically when decoding the token. See the issue for details.

Adding a test that checks if tokens generated using structs as content can be validated.
@FossoresLP FossoresLP self-assigned this Dec 1, 2018
@coveralls
Copy link

Pull Request Test Coverage Report for Build 28

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 26: 0.0%
Covered Lines: 162
Relevant Lines: 162

💛 - Coveralls

@FossoresLP FossoresLP changed the title WIP: Fix struct token validation Fix struct token validation Dec 3, 2018
@FossoresLP FossoresLP merged commit bd96544 into master Dec 3, 2018
@FossoresLP FossoresLP deleted the struct-token-validation-fail branch December 3, 2018 20:30
@FossoresLP FossoresLP changed the title Fix struct token validation Add test for struct token validation issue Dec 24, 2018
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.

None yet

2 participants