-
Notifications
You must be signed in to change notification settings - Fork 0
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
Security Concern #1
Comments
Thanks for telling I actually wanted to submit an assignment via github
i'll remove it
…On Sat, 16 Jul, 2022, 3:06 pm Rohan Kulkarni, ***@***.***> wrote:
- In script.js client id is visible which is auth token for accessing
the API that should not be leaked. Please remove the client id from file.
- You can add that in env and use it from there !!!
Thanks
https://github.com/misbaa/Unsplash-API/blob/b353fb4effefb90fddc3627f7b4bea87ac1983e4/Unsplash-API/script.js#L1
—
Reply to this email directly, view it on GitHub
<#1>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AQJT4KROVZCKSQRCRLQOWMTVUJ7C5ANCNFSM53XZ6WDA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Great !! 👍
…On Sat, Jul 16, 2022, 5:02 PM misbaa ***@***.***> wrote:
Thanks for telling I actually wanted to submit an assignment via github
i'll remove it
On Sat, 16 Jul, 2022, 3:06 pm Rohan Kulkarni, ***@***.***>
wrote:
>
> - In script.js client id is visible which is auth token for accessing
> the API that should not be leaked. Please remove the client id from file.
> - You can add that in env and use it from there !!!
>
> Thanks
>
>
>
https://github.com/misbaa/Unsplash-API/blob/b353fb4effefb90fddc3627f7b4bea87ac1983e4/Unsplash-API/script.js#L1
>
> —
> Reply to this email directly, view it on GitHub
> <#1>, or unsubscribe
> <
https://github.com/notifications/unsubscribe-auth/AQJT4KROVZCKSQRCRLQOWMTVUJ7C5ANCNFSM53XZ6WDA
>
> .
> You are receiving this because you are subscribed to this thread.Message
> ID: ***@***.***>
>
—
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ASIJKM7Q6K4TL6PJPTWXSU3VUKMVJANCNFSM53XZ6WDA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Thanks
Unsplash-API/Unsplash-API/script.js
Line 1 in b353fb4
The text was updated successfully, but these errors were encountered: