-
Notifications
You must be signed in to change notification settings - Fork 3
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
Netlify URLs asking for password #3
Comments
I've enabled this password, as someone started to fetch data from the site every second and consumed all the site's bandwidth + an additonal data package. I'm not willing to pay for so much data usage for something so trivial, so I'll likely kill the site and drop the daily exports functionality for the time being. I know this sucks, but it's also insane that someone would be downloading 500gb of ClojureDocs data a month. Clearly that's some malicious usage of the site and it's not something I can easily restrict either. The damn script has generated 10 gigs of bandwidth just by re-rendering constantly the 301 page.... |
@liquidz Let me know if you have any ideas on the subject - I was thinking of either of maybe just serving the exports from the Git repo, as we won't have to deal with bandwidth limits this way. |
Hi Bozhidar! Is it possible to make CIDER to load this file from the local filesystem rather then Netlify as a workaround? |
Everything is possible. :-) I liked the idea of the central place with a fresh snapshot, as this was the most convenient experience from a user standpoint. Btw, I have to say I'm surprised someone uses the refresh functionality, as the data in ClojureDocs doesn't change that often anyways. |
Well, I'm just using spacemacs with the default CIDER setup. Autocompletion and doc functionality stopped working for me and I saw lots of 403 errors in repl. So here is how I got here in the first place :) |
@coder11 What's your CIDER version? Normally CIDER won't interact with this site at all unless you request a manual update of the ClojureDocs cache, as Orchard bundles a copy of the export. |
(cider-version) gives me this: |
@coder11 Yeah, indeed. The old versions used to fetch this data when starting, which I assume is the problem you encountered. You should update to 1.0. |
Seems it's going to be easy to trigger a daily job with GHA (https://docs.github.com/en/free-pro-team@latest/actions/reference/events-that-trigger-workflows#schedule) and have it commit the fresh exports to the repo. |
I've moved the exports under the repo, for whoever needs them. I'll get to automating the export process down the road, unless @liquidz beats me to it. I don't care that much about having fresh data, so it's not a big priority for me and I've very short on time these days. I'll kill the old site completely as well. All the useful information from it is now in the README. |
@bbatsov I've tried automatic commit using GitHub Actions in the following repositories, so I will automate the update on the repository. |
@liquidz Thanks! Much appreciated! 🙇♂️ |
See clojure-emacs/clojuredocs-export-edn#3 for details.
I don't think anyone uses this as their workflow. I had a colleague of mine for whom the CIDER started breaking with errors on nrepl due to this. So I reported it. I don't think this causes any issues with newer version of CIDER and middlewares. An upgrade should resolve the issue for anyone facing same thing. A different workaround to stop middleware from doing auto refresh, I did |
https://clojuredocs-edn.netlify.app/export.compact.edn
https://clojuredocs-edn.netlify.app/export.edn
These URLs are asking for password.
This breaks thing in cider-nrepl clojuredocs middleware when it tries to refresh the cache.
Has anything changed in the netlify deployment?
The text was updated successfully, but these errors were encountered: