Skip to content

Conversation

@weijunyu
Copy link
Contributor

Description

Separated JOSE and signature validator into separate states.

Fixes # (issue number)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • Code enhancement and update (non-breaking change)

How Has This Been Tested?

Manually using test files provided.

Moved mainController to signatureValidatorController.
Moved controllers and associated templates to features/ folder.
Controllers and templates are now imported using webpack instead of angular DI.
Aligned styles between signature and JOSE validator.
@weijunyu weijunyu requested a review from robincher March 29, 2018 00:45
@robincher robincher added the enhancement New feature or request label Mar 29, 2018
Copy link
Contributor

@robincher robincher left a comment

Choose a reason for hiding this comment

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

Can update the readme.md with the new screenshots

Cancel
</button>
</div>
`
Copy link
Contributor

Choose a reason for hiding this comment

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

Can remove this orphan characters also.

Copy link
Contributor

@robincher robincher left a comment

Choose a reason for hiding this comment

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

Additionally, just ensure the missing semi colons etc are added and bring down the code quality issues that are avoidable.

@GovTechSG GovTechSG deleted a comment Mar 29, 2018
@GovTechSG GovTechSG deleted a comment Mar 29, 2018
@GovTechSG GovTechSG deleted a comment Mar 29, 2018
@GovTechSG GovTechSG deleted a comment Mar 29, 2018
@robincher
Copy link
Contributor

Hi @weijunyu ,

Before you merge, remember to update the ChangeLog and commit it.

README.md Outdated

### Decrypting JWE
![jose_jwe.png](https://github.com/GovTechSG/apex-signature-validator/blob/master/assets/jose_jwe.png)
![jose_jwe.png](https://github.com/GovTechSG/apex-signature-validator/blob/master/assets/jose_jwe_verified.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

Broken Image link

Should be ![jose_jwe_verified.png](https://github.com/GovTechSG/apex-signature-validator/blob/master/assets/jose_jwe_verified.png)

README.md Outdated

### Verifying JWS
![jose_jws.png](https://github.com/GovTechSG/apex-signature-validator/blob/master/assets/jose_jws.png)
![jose_jws.png](https://github.com/GovTechSG/apex-signature-validator/blob/master/assets/jose_jws_verified.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

Broken Image link

Should be like this

![jose_jws_verified.png](https://github.com/GovTechSG/apex-signature-validator/blob/master/assets/jose_jws_verified.png)

Updated var declarations and code style.
@weijunyu weijunyu merged commit 6696054 into master Mar 29, 2018
@GovTechSG GovTechSG deleted a comment Mar 29, 2018
@weijunyu weijunyu deleted the feature/state-separation branch March 29, 2018 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants