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

[Bug][fn] Go functions runtime does not support any sort of auth. #20451

Closed
2 tasks done
flowchartsman opened this issue Jun 1, 2023 · 0 comments · Fixed by #20468 · May be fixed by flowchartsman/pulsar#6
Closed
2 tasks done

[Bug][fn] Go functions runtime does not support any sort of auth. #20451

flowchartsman opened this issue Jun 1, 2023 · 0 comments · Fixed by #20468 · May be fixed by flowchartsman/pulsar#6
Labels
area/function type/bug The PR fixed a bug or issue reported a bug

Comments

@flowchartsman
Copy link
Contributor

flowchartsman commented Jun 1, 2023

Search before asking

  • I searched in the issues and found nothing similar.

Version

After enabling JWT auth on our cluster, all of our Go functions started failing. This is because authentication information is only passed to the pulsar-admin command, and also because the Go function SDK has no concept of auth providers.

Minimal reproduce step

Just enable auth, and watch your go functions start to complain:

broker.pulsar.svc.cluster.local:6650 local_addr=172.27.168.187:37436
2023/06/01 03:05:16.139 connection.go:226: [warning] Failed to establish connection with broker: 'Unable to authenticate'  remote_addr=pulsar://pulsar-broker.pulsar.svc.cluster.local:6650 local_addr=172.27.168.187:37436
2023/06/01 03:05:45.027 log.go:77: [error] create producer error:connection error
2023/06/01 03:05:45.027 instance.go:210: [fatal] connection error
Stream closed EOF for pulsar/pf-public-default-function-test-30-c169f141-0 (pulsarfunction)

What did you expect to see?

Working functions.

What did you see instead?

Dead functions.

Anything else?

No response

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@flowchartsman flowchartsman added the type/bug The PR fixed a bug or issue reported a bug label Jun 1, 2023
flowchartsman added a commit to flowchartsman/pulsar that referenced this issue Jun 1, 2023
This is the intial checkin of the work on apache#20451, currently
supporting org.apache.pulsar.client.impl.auth/AuthenticationToken only.
flowchartsman added a commit to flowchartsman/pfunc that referenced this issue Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/function type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
2 participants