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

Multiline values in ini files broken #3705

Closed
nilmerg opened this issue Mar 4, 2019 · 5 comments · Fixed by #3706
Closed

Multiline values in ini files broken #3705

nilmerg opened this issue Mar 4, 2019 · 5 comments · Fixed by #3706
Assignees
@nilmerg
Labels
area/configuration Affects the configuration area/framework Affects third party integration/development bug Something isn't working ref/NC
Milestone
2.6.3

Comments

@nilmerg
Copy link
Member

nilmerg commented Mar 4, 2019

Since 86ae8c1 multiline values in ini files cause the UI to badly crash.

There's at least one native way (announcements) with which to get newlines into ini files using the UI.
@nilmerg nilmerg added bug Something isn't working area/framework Affects third party integration/development area/configuration Affects the configuration labels Mar 4, 2019
@nilmerg nilmerg added this to the 2.7.0 milestone Mar 4, 2019
@nilmerg nilmerg self-assigned this Mar 4, 2019
nilmerg added a commit that referenced this issue Mar 4, 2019
@nilmerg
config: Escape and unescape line breaks in ini values
769d490 
refs #3705
@nilmerg nilmerg mentioned this issue Mar 4, 2019
Merged
@widhalmt
Copy link
Member

widhalmt commented Mar 5, 2019

ref/NC/602603

@widhalmt
Copy link
Member

widhalmt commented Mar 5, 2019

ref/NC/603991

@nilmerg nilmerg mentioned this issue Apr 5, 2019
Closed
@ekeih
Copy link

ekeih commented Apr 5, 2019

@nilmerg Thanks for fixing several of the ini-file issues! Currently this blocks our update to 2.6.2.
Do you have any idea when you will release 2.7.0? Or would it be possible to release a bugfix release 2.6.3 to fix the ini-file issues from 2.6.2?
Either way: Thank you for fixing it! :)

@nilmerg
Copy link
Member Author

nilmerg commented Apr 8, 2019

@ekeih 2.7 is still way off. A 2.6.3 is more likely, though not planned yet. But that's currently on schedule and an appropriate milestone may be created soon.

@ekeih
Copy link

ekeih commented Apr 15, 2019

@nilmerg Thanks for the feedback! :)

@nilmerg nilmerg modified the milestones: 2.7.0, 2.6.3 Apr 17, 2019
@nilmerg nilmerg added the ref/NC label May 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nilmerg nilmerg

Labels
area/configuration Affects the configuration area/framework Affects third party integration/development bug Something isn't working ref/NC
Projects
None yet
Milestone
2.6.3
Development

Successfully merging a pull request may close this issue.

Fix multiline values in ini files broken Icinga/icingaweb2
3 participants
@widhalmt @ekeih @nilmerg