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

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)
그러면, 나중에 인증 토큰을 요구할 때 쯤에 다시 고려하실 건가요?

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.

CI 추가 요청: jsDelivr-Purge
2 participants