Skip to content

Feature: Add multiple configuration lines with 'BackendExtra' #85

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.

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

Closed
aisbergg opened this issue Nov 19, 2018 · 10 comments
Closed

Feature: Add multiple configuration lines with 'BackendExtra' #85

aisbergg opened this issue Nov 19, 2018 · 10 comments
Assignees

Comments

@aisbergg
Copy link
Contributor

It would be nice to add multiple lines to the backend with BackendExtra. Like other parameters the string should be separated with a comma.

@thomasjpfan
Copy link
Contributor

Currently, BackendExtra does not support the comma to new-line notation. This may be because there are some haproxy options that use comma, such as resolve-opts: https://cbonte.github.io/haproxy-dconv/1.8/configuration.html#5.2-resolve-opts

@dimovnike
Copy link

This would be definitely great to have. I also need this option.

@chrsch
Copy link

chrsch commented Dec 28, 2018

I also would love to see this.

@thomasjpfan
Copy link
Contributor

My only concern with adding this feature using comma as a separator is that it will break backwards compatibility. For example:

http-request set-header x-path %[hdr(x-path),regsub(/+,/,g)]

In this case, the comma needs to be there.

@chrsch
Copy link

chrsch commented Dec 29, 2018

I see. Two quick options then came to my mind earlier this day:

@thomasjpfan
Copy link
Contributor

We could give the user complete control by adding BackendExtraMultilineSeparator.

@chrsch
Copy link

chrsch commented Dec 29, 2018

Oh yes I like this idea.

@vfarcic
Copy link
Contributor

vfarcic commented Mar 2, 2019

This project needs adoption. I moved to Kubernetes and cannot dedicate time to this project anymore. Similarly, involvement from other contributors dropped as well. Please consider contributing yourself if you think this project is useful.

@lle0x
Copy link
Collaborator

lle0x commented Mar 31, 2022

Dear @aisbergg

If this issue is still relevant, please feel free to leave a comment here.

@lle0x lle0x self-assigned this Mar 31, 2022
@aisbergg
Copy link
Contributor Author

I moved to Traefik

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

No branches or pull requests

6 participants