-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
Fix node security link #7081
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
Fix node security link #7081
Conversation
* commit 'ccb045b68c5b4d983a90fa125513fc476e4e2387': fix: upgrade @graphql-tools/links from 6.2.4 to 6.2.5 (parse-community#7007) fix: upgrade pg-promise from 10.7.0 to 10.7.1 (parse-community#7009) fix: upgrade jwks-rsa from 1.10.1 to 1.11.0 (parse-community#7008) fix: upgrade graphql from 15.3.0 to 15.4.0 (parse-community#7011) update stale bot (parse-community#6998) fix(beforeSave/afterSave): Return value instead of Parse.Op for nested fields (parse-community#7005) fix(beforeSave): Skip Sanitizing Database results (parse-community#7003) Fix includeAll for querying a Pointer and Pointer array (parse-community#7002) Init (parse-community#6999)
Codecov Report
@@ Coverage Diff @@
## master #7081 +/- ##
==========================================
+ Coverage 93.63% 93.90% +0.26%
==========================================
Files 169 169
Lines 12498 12535 +37
==========================================
+ Hits 11703 11771 +68
+ Misses 795 764 -31
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Changed to draft due to ongoing discussion about form. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
@davimacedo could you add the DNS record for this?
Do you want me to create a DNS record from report.parseplatform.org to https://form.jotform.com/210205133587044, right? I am not sure if that's possible though. A CNAME from report.parseplatform.org to form.jotform.com is possible for sure, but you need also to check with form.jotform.com if you need to setup something in their side. I can also create a permanent redirect from https://report.parseplatform.org to https://form.jotform.com/210205133587044. |
Yes, a redirect, sorry, custom domain feature is disabled in JotForm free version. |
Done. Could you please check it out? |
Works fine! I think we can merge this one? |
Yes. I think we can. |
I think someone should try out the form actually, whether it sends an email, I don't have access to the email account community@... And if anyone wants to install the mobile app to get push notifications on form submits, I can give the Jotform credentials. |
Just tested, email received |
* removed node security link * improve PGP wording * Added report form link * removed PGP key
🎉 This change has been released in version 5.0.0-beta.1 |
🎉 This change has been released in version 5.0.0 |
New Pull Request Checklist
Issue Description
Node security link is outdated, npm does no longer triage vulnerabilities.
Related issue: closes parse-community/docs#796
Approach
Link removed.
TODOs before merge:
report.parseplatform.org
DNS entry, see 📙 Fix Node Security link docs#796 (comment)