temporalio / temporalio/samples-go
[Bug] serverjwtauth sample doesn't work with newer servers
Open
Beginner friendly
Nobody has claimed this yet.
bug
- Dominant language
- Go
- Stars
- 749
- Forks
- 248
- Avg merge
- 4d 2h
- Merged PRs (30d)
- 8
Description
Describe the bug
Multiple server changes were made since this sample was written. Specifically we need to:
- Update README to have them set
USE_INTERNAL_FRONTEND=trueenv var andSERVICES=frontend:internal-frontend:history:matching:worker - Update https://github.com/temporalio/samples-go/blob/6b93fe45a8824a56ac70f30832d262b94921a80d/serverjwtauth/key/main.go#L95C35-L95C35 to change
system:admintotemporal-system:admin
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Update the serverjwtauth README with the USE_INTERNAL_FRONTEND=true and SERVICES values specified in the issue. In key/main.go, inspect the referenced line and change system:admin to temporal-system:admin. Done means the sample instructions match newer servers and the JWT authorization uses the updated role.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- authentication
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 65/100