Skip to content

Conversation

@glokta1
Copy link
Contributor

@glokta1 glokta1 commented Apr 24, 2023


PR description

Obfuscate leftover emails using advanced munging. Advanced munging converts each char to its ASCII value. This technique is purported to be effective against 85% of web crawlers.

Technique described here: https://www.projecthoneypot.org/how_to_avoid_spambots_2.php

Aesthetically, it looks better than the basic approach of replacing @ and . with (at) and (dot) while also being more effective. Less effective than using JS to obfuscate but has the benefit of being visible without loading any JS.

NOTE: I can't get the project setup for now so I couldn't test this change in the browser.

Advanced munging converts each char to its ASCII value. Technique described here: https://www.projecthoneypot.org/how_to_avoid_spambots_2.php
@netlify
Copy link

netlify bot commented Apr 24, 2023

Deploy Preview for ssgsa-dev ready!

Name Link
🔨 Latest commit
🔍 Latest deploy log https://app.netlify.com/sites/ssgsa-dev/deploys/6447c99e86a48560c7fa7b90
😎 Deploy Preview https://deploy-preview-232--ssgsa-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@pnijhara
Copy link
Collaborator

@glokta1 your approach looks good to me.
@himanshuvarandani I don't have the project setup with me, can you review this?

@glokta1
Copy link
Contributor Author

glokta1 commented Apr 30, 2023

Any update on this? @himanshuvarandani

@pnijhara
Copy link
Collaborator

pnijhara commented May 12, 2023

@glokta1 I think @himanshuvarandani is busy with his office work. I checked the log and saw that your changes are not getting rendered by the Netlify build. For now, you can update the email addresses such as contact@ssgsa.us to contact [at] ssgsa [dot] us (including spaces) and also, remove the tag and use the

tag instead. We can discuss your changes when Himanshu gets back online.

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.

2 participants