search

jeffhollan / LogicAppTemplateCreator

Script to convert Logic Apps into templates for deployment
MIT License
143 stars 74 forks source link

Implement clear parameter value #57

Closed Splaxi closed 4 years ago

Splaxi commented 5 years ago

To support the scenario where you want to be 100% sure that you don't leak sensitive / semi-sensitive data from your development.

Will simply assign an empty "[]" json property value to the defaultvalue.

Contains: 1 x Unit test to test that it works like expected

1 x modification of existing unit test, to validate that the change didn't break anything.

Splaxi commented 4 years ago

My initial testing found that [] worked better.

If you want, I can see if I can do some more testing with the other values...

MLogdberg commented 4 years ago

Then we go for []. Thanks for testing it out

Splaxi commented 4 years ago

And if we get some bug reports, then we know where to look 😉

MLogdberg commented 4 years ago

True!

Med vänlig hälsning, Mattias

Skickat från min iPhone

9 aug. 2019 kl. 20:13 skrev Mötz Jensen notifications@github.com<mailto:notifications@github.com>:

And if we get some bug reports, then we know where to look 😉

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHubhttps://github.com/jeffhollan/LogicAppTemplateCreator/pull/57?email_source=notifications&email_token=AD4IHOUNRU2MNTEF2WCLTV3QDWXTRA5CNFSM4H6UGRRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD37MVEY#issuecomment-520014483, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AD4IHOTM22XXZU6AQLRTGSTQDWXTRANCNFSM4H6UGRRA.