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

Version string in html header ... #1457

Closed
agaleski opened this issue May 25, 2020 · 4 comments
Closed

Version string in html header ... #1457

agaleski opened this issue May 25, 2020 · 4 comments

Comments

@agaleski
Copy link

I just found this string by accident on our staging in the HTML header:
<!-- This site has installed PayPal for WooCommerce v2.2.3 - https://www.angelleye.com/product/woocommerce-paypal-plugin/ -->

I understand that you might have reasons to add something "crawlable" to the header to identify who is using your product. But exposing the version number with eventually known bugs/exploits is an absolute nogo for me.

Please either add the option to not print the version number OR remove it entirely.

Thank you.

@drewangell
Copy link
Collaborator

Hey, sorry, I just now saw your message here. We do most of our stuff in Jira so I wasn't seeing this.

We can certainly address this one way or another. I'm going to get it in our backlog now.

@angelleye-gitcommits
Copy link

angelleye-gitcommits commented Sep 2, 2020

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


The following Contributor have begun work and are competing for the bounty. .

Below user(s) claimed they can complete the work within defined time period.

Please review their action plans below:

1) deepakmaurya has been approved to work.

Message: Hi Andrew,
I have started working on this issue, I will open the pull request for this change by today.
Let me know if there is anything that we need to consider while we are doing this.
Thanks


This issue has a bounty of $10.00 attached to it.

@angelleye-gitcommits
Copy link

deepakmaurya, has started work on the bounty.


Message: Hi Andrew,
I have started working on this issue, I will open the pull request for this change by today.
Let me know if there is anything that we need to consider while we are doing this.
Thanks

Click Here to view more on GitCommits Bounty detail page.

@angelleye-gitcommits
Copy link

deepakmaurya Work has been approved by drewangell.


Message: Pull request has been approved, you will receive the payment shortly.

Click Here to view more on GitCommits Bounty detail page.

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

No branches or pull requests

4 participants