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.

Sign up for GitHub

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

Jump to bottom

Wrong warning about public key permissions #1118

Open
jkufner opened this issue May 29, 2020 · 2 comments
Open

Wrong warning about public key permissions #1118

jkufner opened this issue May 29, 2020 · 2 comments

Comments

@jkufner
Copy link

jkufner commented May 29, 2020

PHP Notice: Key file "file:///.../public/../public.key" permissions are not correct, recommend changing to 600 or 660 instead of 644 in /.../vendor/league/oauth2-server/src/CryptKey.php on line 63

This warning makes no sense. It is a public key. Anybody can have it.
@Sephster
Copy link
Member

Fair point. Would welcome a PR to fix this.

@Sephster
Copy link
Member

Sephster commented Jul 3, 2020

Looking at this further, the CryptKey class was originally introduced to use passwords with your private key. I need to do a bit more investigation into this to find the best way to resolve it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jkufner @Sephster