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

Implemented Login/Sign Up page #64

Closed
wants to merge 0 commits into from

Conversation

KARTIK-RAINA5802
Copy link

No description provided.

@vercel
Copy link

vercel bot commented Aug 16, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
memories-app ❌ Failed (Inspect) Aug 17, 2022 at 11:14AM (UTC)
memories-app-server ✅ Ready (Inspect) Visit Preview Aug 17, 2022 at 11:14AM (UTC)

@gitguardian
Copy link

gitguardian bot commented Aug 16, 2022

⚠️ GitGuardian has uncovered 5 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request
GitGuardian id Secret Commit Filename
- MongoDB Credentials 873f26e server/index.js View secret
- MongoDB Credentials 873f26e server/index.js View secret
- MongoDB Credentials cd2ca45 server/index.js View secret
- MongoDB Credentials cd2ca45 server/index.js View secret
- MongoDB Credentials 916d909 server/index.js View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secrets safely. Learn here the best practices.
  3. Revoke and rotate these secrets.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!

@KARTIK-RAINA5802
Copy link
Author

Ok, I found out that there were some errors due to which deployment failed and I solved those errors and it's working fine now on my system but GitGuardian is giving some errors related to MongoDB credentials which I am not able to solve. Can anyone please help with this issue?

And yes I have resolved merge conflicts and did npm I also before pushing.

@shreya024
Copy link
Owner

Ok, I found out that there were some errors due to which deployment failed and I solved those errors and it's working fine now on my system but GitGuardian is giving some errors related to MongoDB credentials which I am not able to solve. Can anyone please help with this issue?

And yes I have resolved merge conflicts and did npm I also before pushing.

Great work on resolving the merge conflicts.Don't worry about the MongoDB credentials thing but the client side deployment is failing.
image

@shreya024 shreya024 linked an issue Aug 16, 2022 that may be closed by this pull request
@KARTIK-RAINA5802
Copy link
Author

Yes I worked on that issue and solved the errors and it started working. So may I push the changes and create a new PR ?

@shreya024
Copy link
Owner

Yes I worked on that issue and solved the errors and it started working. So may I push the changes and create a new PR ?

Just push the changes,it'll reflect here only.No need to make a new PR

@KARTIK-RAINA5802
Copy link
Author

Can you send me vercel error because I am not able to see any error on my local system and it's working fine.
I also connected it to my MongoDB compass to check whether login/signup is working or not and it was working.

Screenshot (1381)

@KARTIK-RAINA5802
Copy link
Author

and I also tried to check the vercel details but it's showing 404 page not found.

@shreya024
Copy link
Owner

and I also tried to check the vercel details but it's showing 404 page not found.

This is the vercel error
image

@KARTIK-RAINA5802
Copy link
Author

Can anyone help me with this issue? I am not able to find out errors locally which is making it more difficult for me to figure out what's going wrong.

And earlier I closed my PR which passed the two checks and then I just pulled the changes and created this new PR and it's not able to pass client-side deployment. I don't know what happened.

@shreya024
Copy link
Owner

Can anyone help me with this issue? I am not able to find out errors locally which is making it more difficult for me to figure out what's going wrong.

And earlier I closed my PR which passed the two checks and then I just pulled the changes and created this new PR and it's not able to pass client-side deployment. I don't know what happened.

You can tag the Project Mentors and fellow contributors in discord for help.You could also try deploying your branch on your vercel to figure out the error :)

@prakhar728
Copy link

prakhar728 commented Aug 17, 2022

Can anyone help me with this issue? I am not able to find out errors locally which is making it more difficult for me to figure out what's going wrong.

And earlier I closed my PR which passed the two checks and then I just pulled the changes and created this new PR and it's not able to pass client-side deployment. I don't know what happened.

Okay I am not sure. But did you add .env file and Environment variables on the frontend? With React you need to name the variables as REACT_APP_YOUR_VARIABLE.

  1. Try the npm run build command in your localhost and see if it's working.

@KARTIK-RAINA5802
Copy link
Author

I guess It is build successfully right ?

image

@KARTIK-RAINA5802
Copy link
Author

And I am also getting merge conflicts should I resolve them first?

@shreya024
Copy link
Owner

And I am also getting merge conflicts should I resolve them first?

Yes

@shreya024
Copy link
Owner

@KARTIK-RAINA5802 Resolve the conflicts and complete the deployment fast so we can merge.Else this issue will be up for reassignment

@soumali28
Copy link
Collaborator

@KARTIK-RAINA5802 any updates?? please resolve the conflicts so that we can merge the pr

@KARTIK-RAINA5802
Copy link
Author

KARTIK-RAINA5802 commented Sep 22, 2022 via email

@fykaa
Copy link
Collaborator

fykaa commented Sep 23, 2022

Thanks a lot for your contributions @KARTIK-RAINA5802 . We are happy that you tried. Unassigning you for now, you could still work on this issue later, All the best!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement LogIn/Sign Up
5 participants