search

labbsr0x / whisper

A cloud-native Identity and OAuth Provider implemented with Golang and ORY Hydra
MIT License
12 stars 4 forks source link

Issue#53 - Update Hydra's environment variables #54

Closed claudiosegala closed 4 years ago

claudiosegala commented 4 years ago

Fix #53, start #12 and some minor improvements.

claudiosegala commented 4 years ago

@abilioesteves do you know any way to stress test every variable?

eabili0 commented 4 years ago

@claudiosegala, what do you mean?

claudiosegala commented 4 years ago

@claudiosegala, what do you mean?

I mean that I change the variables and from what I saw, it seemed to work. But I do not think that the tests I made covered the usages of every variable that I changed.

eabili0 commented 4 years ago

@claudiosegala, I think we should open a new issue to discuss how to test all the important variables (so Whisper can work nicely with Hydra). It's a complicated subject, since we are talking about integration testing (not unit testing). We should then discuss how to proceed with the creation of an Integration Testing pipeline, and then stress-out these variables.

claudiosegala commented 4 years ago

@abilioesteves, I agree with you. Should I open this issue?

eabili0 commented 4 years ago

@claudiosegala, yes, please