cdbartholomew opened a new pull request #9411:
URL: https://github.com/apache/pulsar/pull/9411
Fixes #8091
### Motivation
In order to use authentication in Presto (and Pulsar SQL), the password
authenticators plugin must be present in the distribution. If you configure
password authentication and the plugin is not present, the server will not
start.
### Modifications
This update pulls in the password authenticators plugin from the PrestoSQL
build and packages as part of the presto distribution used by the `pulsar
sql-worker run` command.
### Verifying this change
I have built a Docker image with this change and confirmed that file-based
password authentication works with this change. Since the plugin comes from the
Presto build, I don't think we need any extra tests for it. Adding an
integration test would be difficult because configuring authentication in
Presto requires HTTPS to also be set up.
### Does this pull request potentially affect one of the following parts:
*If `yes` was chosen, please highlight the changes*
- Dependencies (does it add or upgrade a dependency): yes. Adds a new
dependency on the Presto password plugin.
- The public API: no
- The schema: no
- The default values of configurations: no
- The wire protocol: no
- The rest endpoints: no
- The admin cli options: no
- Anything that affects deployment: no
### Documentation
The plugin can be configured by modifying the files in the
`/etc/conf/presto` directory by following the PulsarSQL (now Trino)
documentation. For example, this
[link](https://trino.io/docs/current/security/password-file.html#) explains has
to configure file-based password authentication.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
