Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

CI 추가 요청: jsDelivr-Purge #385

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

piquark6046
Copy link
Contributor

Fix #384

@yous
Copy link
Owner

yous commented Nov 26, 2023

제안 감사합니다. https://www.jsdelivr.com/tools/purge 요청을 자동으로 해주는 거라고 생각하면 될 것 같은데, https://github.com/jsdelivr/jsdelivr#purge-cache 여기 설명 보면 결국 rate limit이 있고 해서요.

GitHub URL 그대로 사용하는 것에 비교해서 jsdelivr 매번 purge 하는 게 어느 정도 이점이 있을지 잘 모르겠네요. 필터 자체 용량이 그리 크지도 않고요. 일반적으로는 버전 배포용으로 jsdelivr를 사용하다 보니까요. 일단은 좀 더 지켜보도록 하겠습니다.

@piquark6046
Copy link
Contributor Author

https://github.com/hagezi/dns-blocklists 에서도 추가되긴 했습니다. (https://github.com/hagezi/dns-blocklists/discussions/1852)
그러면, 나중에 인증 토큰을 요구할 때 쯤에 다시 고려하실 건가요?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI 추가 요청: jsDelivr-Purge
2 participants