add-scope-to-token #33

tobiasbp · 2023-10-24T12:02:49Z

tobiasbp commented

2023-10-24 12:02:49 +00:00

This PR adds the ability to set scopes for tokens (they can not be used for much without). Removed the username from the token resource as the owner can not be configured, as it will be owned by the user creating the resource.

As far as I can tell, it's not possible to modify the scopes for a existing token using the API, so a token created by the provider will be recreated if the list of scopes is updated. This reflects what is possible using the GUI.

This PR fixes this issue: #32

This PR adds the ability to set scopes for tokens (they can not be used for much without). Removed the _username_ from the _token resource_ as the owner can not be configured, as it will be owned by the user creating the resource. As far as I can tell, it's not possible to modify the scopes for a existing token using the API, so a token created by the provider will be recreated if the list of scopes is updated. This reflects what is possible using the GUI. This PR fixes this issue: https://gitea.com/gitea/terraform-provider-gitea/issues/32

tobiasbp added 10 commits 2023-10-24 12:02:49 +00:00

Add ability to set scopes for resource gitea_token c9def90a50

Add scopes to example for gitea_token 4579f52617

Remove username from gitea_token as the owner of the token can not be set, as implied 0b2a6695de

Fix: Ignore order of scopes when creating diff by using TypeSet instead of TypeList fd255a9825

Remove out commented import of sort 4a9bc54879

Add map of vaild scopes. Fix wrong assert when looping trough scopes (now a set) 2be756d96e

Add check for validity of scopes used with tokens 6c1987c76f

Minor refactor of casting? to string in token scopes df5ce127c7

Refactor of options to CreateAccessToken() 5951741f66

Fix: Don't use snake case for variable name

Setup Terraform / Terraform Versions (pull_request) Has been cancelled