Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support for config maps with keys that have embedded '.' #859

Closed
dmlloyd opened this issue Feb 12, 2019 · 3 comments
Closed

Support for config maps with keys that have embedded '.' #859

dmlloyd opened this issue Feb 12, 2019 · 3 comments
Milestone
0.9.0

Comments

@dmlloyd
Copy link
Member

dmlloyd commented Feb 12, 2019

We don't presently support configuration maps whose keys contain embedded . characters. NameIterator should be update to either support quoted segments (e.g. shamrock.foo."bar.baz".zap) or to use backslash to escape the embedded . (e.g. shamrock.foo.bar\.baz.zap).

Opinions?
@gsmet
Copy link
Member

gsmet commented Feb 12, 2019

I would vote for quotes.

@cescoffier
Copy link
Member

Quote as well....

@dmlloyd dmlloyd added this to the 0.9.0 milestone Feb 12, 2019
@dmlloyd
Copy link
Member Author

dmlloyd commented Feb 12, 2019

Quotes it is.

maxandersen pushed a commit to maxandersen/quarkus that referenced this issue Nov 5, 2022
@quintesse
feat: info tools now returns a lot more information (quarkusio#859)
e03d6d9 
* feat: `info tools` now returns a lot more information

Fixes quarkusio#833

* fix: Make sure we don't get stuck in endless loops

For quarkusio#833

* chore: disable HTML escaping for the `info tools` output
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.9.0
Development

No branches or pull requests
3 participants
@dmlloyd @cescoffier @gsmet