Skip to content

Conversation

@Adamj1232
Copy link
Contributor

Description

Move image to a spot where it is available through an absolute path, add robot.txt, update meta tags

…velop

t merge develop# Please enter a commit message to explain why this merge is necessary,
@Adamj1232 Adamj1232 requested a review from leightkt April 30, 2024 16:07
@socket-security
Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@algolia/[email protected] None +5 258 kB shortcuts
npm/@babel/[email protected] None 0 23.3 kB nicolo-ribaudo
npm/@babel/[email protected] None 0 64.6 kB nicolo-ribaudo
npm/@babel/[email protected] None 0 52 kB nicolo-ribaudo
npm/@babel/[email protected] None 0 31.6 kB nicolo-ribaudo
npm/@babel/[email protected] None 0 11.7 kB nicolo-ribaudo
npm/@babel/[email protected] None 0 18.4 kB nicolo-ribaudo
npm/@babel/[email protected] None 0 1.89 MB nicolo-ribaudo
npm/@babel/[email protected] None 0 247 kB nicolo-ribaudo
npm/@babel/[email protected] environment 0 2.41 MB nicolo-ribaudo
npm/@cosmjs/[email protected] None +1 227 kB webmaster128
npm/@cosmjs/[email protected] None 0 139 kB webmaster128
npm/@cosmjs/[email protected] None 0 24.6 kB webmaster128
npm/@cosmjs/[email protected] None 0 36.8 kB webmaster128
npm/@cosmjs/[email protected] None 0 80 kB webmaster128
npm/@cosmjs/[email protected] None 0 30.4 kB webmaster128
npm/@cosmjs/[email protected] network Transitive: environment +3 744 kB webmaster128
npm/@cosmjs/[email protected] None 0 14.5 kB webmaster128
npm/@docsearch/[email protected] None 0 27.9 kB shortcuts
npm/@docsearch/[email protected] Transitive: environment +4 2.12 MB shortcuts
npm/@emotion/[email protected] environment +4 98.5 kB emotion-release-bot
npm/@ethereumjs/[email protected] None 0 2.53 MB holgerd77
npm/@ethereumjs/[email protected] None 0 431 kB holgerd77
npm/@ethereumjs/[email protected] None +6 2.59 MB holgerd77
npm/@iconify-json/[email protected] None 0 859 kB cyberalien
npm/@iconify/[email protected] None 0 25.7 kB cyberalien
npm/@jridgewell/[email protected] None +2 259 kB jridgewell
npm/@lit/[email protected] None 0 785 kB lit-robot
npm/@metamask/[email protected] None 0 150 kB metamaskbot
npm/@ngraveio/[email protected] None +1 138 kB antwnic4
npm/@noble/[email protected] None 0 1.35 MB paulmillr
npm/@noble/[email protected] None 0 747 kB paulmillr
npm/@protobufjs/[email protected] None 0 9.05 kB dcode
npm/@safe-global/[email protected] None 0 33 kB dasanra
npm/@safe-global/[email protected] None +1 242 kB dasanra
npm/@scure/[email protected] None 0 80.4 kB paulmillr
npm/@shapeshiftoss/[email protected] None +3 11.6 MB shapeshift-ci-bot
npm/@solana/[email protected] environment, eval, network +1 11.4 MB buffalojoec
npm/@sveltejs/[email protected] environment 0 8.21 kB svelte-admin
npm/@sveltejs/[email protected] environment, eval +2 810 kB svelte-admin
npm/@svelteness/[email protected] Transitive: filesystem, network +2 17 MB mihar-22
npm/@tailwindcss/[email protected] None 0 86.7 kB bradlc
npm/@toruslabs/[email protected] network +3 9.29 MB chaitanyapotti
npm/@toruslabs/[email protected] None 0 47.9 kB chaitanyapotti
npm/@toruslabs/[email protected] Transitive: environment +6 2.39 MB chaitanyapotti

🚮 Removed packages: npm/@babel/[email protected], npm/@babel/[email protected], npm/@babel/[email protected], npm/@babel/[email protected], npm/@unstoppabledomains/[email protected], npm/@web3-onboard/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected]

View full report↗︎

@socket-security
Copy link

🚨 Potential security issues detected. Learn more about Socket for GitHub ↗︎

To accept the risk, merge this PR and you will not be notified again.

Alert Package NoteSource
Install scripts npm/[email protected]
  • Install script: install
  • Source: npm run rebuild || echo "Keccak bindings compilation fail. Pure JS implementation will be used."
Install scripts npm/@sveltejs/[email protected]
  • Install script: postinstall
  • Source: node postinstall.js

View full report↗︎

Next steps

What is an install script?

Install scripts are run when the package is installed. The majority of malware in npm is hidden in install scripts.

Packages should not be running non-essential scripts during install and there are often solutions to problems people solve with install scripts that can be run at publish time instead.

Take a deeper look at the dependency

Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev.

Remove the package

If you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency.

Mark a package as acceptable risk

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of ecosystem/package-name@version specifiers. e.g. @SocketSecurity ignore npm/[email protected] or ignore all packages with @SocketSecurity ignore-all

@Adamj1232 Adamj1232 merged commit 0184045 into develop Apr 30, 2024
@Adamj1232 Adamj1232 deleted the doc_fix/meta_tags_for_socials branch April 30, 2024 16:16
Adamj1232 added a commit that referenced this pull request Apr 30, 2024
* [DOCS FIX] - Update meta tags for socials image display (#2182)

* Move image to a spot where it is available through an absolute path, add robot.txt, update meta tags
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

Successfully merging this pull request may close these issues.

3 participants