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

User-Generated Tokens Causing Project Config Changes #9

Open
cakleimeier opened this issue Dec 7, 2021 · 1 comment
Open

User-Generated Tokens Causing Project Config Changes #9

cakleimeier opened this issue Dec 7, 2021 · 1 comment

Comments

@cakleimeier
Copy link

I have a site running Craft CMS 3.5.19.1 and Patron 2.4.0. I've noticed something occurring that could cause problems, if I'm understanding the process correctly. As far as I'm able to tell, each time a user authorizes my site with an OAuth login, a Token is generated. That action also creates a new a Project Config YAML file for that new Token in the config/project/plugins/patron/tokens directory.

This would cause un-version-able Project Config changes to occur on the live environment, which I anticipate being difficult to work with. Every time a user authorizes the app on the live site, a new Token file will appear in the Project Config folder, which I will have no way of tracking and which may cause Project Config conflicts.

Is there some way to work around that issue or to disable a new file from being created for each token? Or I am missing something? Should I just gitignore that directory?

Thanks for your help and your time!

@nateiler
Copy link
Contributor

nateiler commented Dec 7, 2021

Hello,

I actually haven't used this in a manner where there are a lot of integration logins; primarily I use a single integration user which results in a token or two. Not to say your use case is incorrect, I just haven't considered it before.

I suppose the best long term approach would be to set a flag to disable the tokens from being managed under project config. I'll need to do some testing around this to make sure they use case works well, but I think that's the best direction.

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