Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

fix GithubIssueTemplate deserialization #990

Merged
merged 17 commits into from
Jun 17, 2021

Conversation

chkeita
Copy link
Contributor

@chkeita chkeita commented Jun 14, 2021

Revert change made in #985
fixed GithubIssueTemplate deserialization

@chkeita chkeita changed the title Investaige secredata serialization Investigate secredata serialization Jun 14, 2021
@chkeita chkeita changed the title Investigate secredata serialization fix GithubIssueTemplate deserialization Jun 16, 2021
@chkeita chkeita marked this pull request as ready for review June 16, 2021 03:13
@bmc-msft
Copy link
Contributor

ADOTemplate and TeamsTemplate are likely impacted by this issue as well for the same reasons. Is it possible to make this a fix within the SecretData implementation rather than one that needs to be replicated for each implementation?

@chkeita
Copy link
Contributor Author

chkeita commented Jun 16, 2021

ADOTemplate and TeamsTemplate are likely impacted by this issue as well for the same reasons. Is it possible to make this a fix within the SecretData implementation rather than one that needs to be replicated for each implementation?

I added two tests for ADOTemplate and TeamsTemplate. They were no affected by the issue .

I could not put the fix in SecretData because it is a generic type and we need to have access to the Generic argument at runtime to do the deserialization.

@chkeita chkeita requested a review from bmc-msft June 16, 2021 16:57
@bmc-msft bmc-msft merged commit cb895d0 into microsoft:main Jun 17, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jul 17, 2021
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.

2 participants