Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Alevsk
Copy link
Contributor

@Alevsk Alevsk commented Apr 7, 2022

  • configuration parse was failing when env value contains multiple =

Signed-off-by: Lenin Alevski [email protected]

@Alevsk Alevsk requested a review from harshavardhana April 7, 2022 06:22
@Alevsk Alevsk self-assigned this Apr 7, 2022
@Alevsk Alevsk requested review from bexsoft, cniackz and dvaldivia April 7, 2022 06:22
dvaldivia
dvaldivia previously approved these changes Apr 7, 2022
@Alevsk Alevsk requested a review from Praveenrajmani April 7, 2022 06:23
Copy link
Member

@harshavardhana harshavardhana left a comment

Choose a reason for hiding this comment

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

Please add a unit test that covers all possible scenarios. Take a look at server side implementation for a more generic parsing logic for config_env

harshavardhana
harshavardhana previously approved these changes Apr 7, 2022
- added unit tests
- uses similar logic to the one used on MinIO

Signed-off-by: Lenin Alevski <[email protected]>
@dvaldivia dvaldivia merged commit 5adaef9 into minio:master Apr 8, 2022
@Alevsk Alevsk deleted the parserawconfiguration-fix branch April 8, 2022 01:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants