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.

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

Http/Https Kamelet: Expose cookieManagementDisabled parameter #2042

Open
oscerd opened this issue May 14, 2024 · 6 comments
Open

Http/Https Kamelet: Expose cookieManagementDisabled parameter #2042

oscerd opened this issue May 14, 2024 · 6 comments
Assignees

Comments

@oscerd
Copy link
Contributor

oscerd commented May 14, 2024

No description provided.

@oscerd
Copy link
Contributor Author

oscerd commented Sep 9, 2024

This will need to create a component instance to expose the parameter. Unless there is a good reason for this, this could be parked for the moment. Creating component instances for a single parameter override, is overkilling.

@oscerd oscerd removed this from the 4.8.0 milestone Sep 9, 2024
@KevinMeng4662
Copy link

@oscerd

We can suspend to expose cookieManagementDisabled parameter.

I sincerely hope that can support below function.
There is actual production use case that through the CamelHttpsSinkConnector to sync the event to RemoteEndpoint.
And the RemoteEndpoint accessing must through mTLS (mutual Transport Layer Security), so need to set javax.net.ssl.keyStore, javax.net.ssl.keyStoreType, javax.net.ssl.keyStorePassword, ssl.KeyManagerFactory.algorithm parameters.

Currently if CamelHttpsSinkConnector does not expose these parameters, that we cannot support this use case.

Thanks a lot.

@oscerd
Copy link
Contributor Author

oscerd commented Sep 12, 2024

What we could do is exposing the useSystemProperties parameters for example https://camel.apache.org/components/4.4.x/http-component.html#_using_system_properties
as first step.
Building sslContextParameters object with YAML could be a bit complex, because you need to define multiple beans to re-use in the sslContextParameters definition, so this would require a separated kamelet for the purpose.

@KevinMeng4662
Copy link

@oscerd

If adopt useSystemProperties parameters to setup CamelHttpsSinkConnector, in one JVM only can run one Connector for specific Keystore, that will waste so much resource.

Although building sslContextParameters object with YAML could be a bit complex, from the user's point of view is still very necessary.

Hope you can make plan for implementing it.

@oscerd
Copy link
Contributor Author

oscerd commented Sep 12, 2024

It won't be in the next release 4.8.0 for sure. I'll see what is needed, you can also have a look yourself at creating a specific kamelet for this and open a PR.

@KevinMeng4662
Copy link

@oscerd I have no backgroud for creating a specific kamelet.

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

2 participants