Skip to content

Moving IDToken in Util as it goes beyond the scope of Search

Komal Makkar requested to merge IdTokenRefactor into master


Moving the IdToken to util, as it will be used by services beyond search (Notification Service will be using it).

Changes include

  • Refactoring
  • Bugfix (non-breaking change that solves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (change that is not backwards-compatible and/or changes current functionality)


  • Added Unit Tests, wherever applicable.
  • Updated the Readme
  • Existing Tests pass: The test shouldSuccessfullySerializeLegalTag is breaking in the existing code.
  • Verified functionality locally
  • Self Reviewed my code for formatting and complex business logic.

Other comments

FYI @dkodeih @danielscholl

Edited by Pavel Bachyla

Merge request reports