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.

Sign up for GitHub

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

Jump to bottom

Deleted #174

Closed
Rexadev opened this issue Sep 23, 2021 · 3 comments
Closed

Deleted #174

Rexadev opened this issue Sep 23, 2021 · 3 comments
Labels
Help wanted Investigating

Comments

@Rexadev
Copy link

Rexadev commented Sep 23, 2021 •
edited

No description provided.

@spirillen
Copy link

why? do you think this should be removed?

@anudeepND
Copy link
Owner

@Rexadev @spirillen we had a similar issue with Moneycontrol, which was fixed in #82 which I fixed by removing:

stat1.moneycontrol.com	
stat2.moneycontrol.com	
stat3.moneycontrol.com	
stat4.moneycontrol.com

And I couldn't find stat.moneycontrol.co.in in my logs while browsing the website and all the features such as charts, live chat, stock prices etc are working fine. Can you please provide more information @Rexadev ?

@anudeepND
Copy link
Owner

Sorry, I couldn't reproduce the issue can you post some screenshots?

you are using .com and i suggested .co.in

stat.moneycontrol.co.in is not in my logs and I could not find it in dev console

anudeepND added a commit that referenced this issue Mar 20, 2022
@anudeepND
Remove stat.moneycontrol.co.in
cca3903 
#174
@Rexadev Rexadev changed the title Remove "stat.moneycontrol.co.in" from blocklist. Deleted May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Help wanted Investigating
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@anudeepND @spirillen @Rexadev