fix: patch python-Wappalyzer dependency for Python 3.12+ compatibility#4
Open
mirusu400 wants to merge 3 commits into
Open
fix: patch python-Wappalyzer dependency for Python 3.12+ compatibility#4mirusu400 wants to merge 3 commits into
mirusu400 wants to merge 3 commits into
Conversation
Owner
|
안녕하세요, 이 문제에 대해 저도 사용하면서 고민을 하고 있었던 부분인데 현업이 바빠져서 아직 해결하지 못했습니다ㅠ.ㅠ 오늘 쭉 검토 해본 결과, 말씀해주신 방향도 좋지만 장기적으로는 기술 스택 식별을 subsurfer가 직접하는게 좋지 않을까라는 생각도 들기도 해서 고려사항 중 하나로 포함해서 조금 더 길게 보려고 합니다. 우선 다른 모듈로 대체하여 사용될 수 있게 수정해서 버전 업데이트를 할 예정입니다. 도커 부분도 검토해보겠습니다. 감사합니다. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
현재 버전에선 python-Wappanalyzer 를 사용하고 있고, 여기서
pkg_resources라이브러리를 사용하고있습니다해당 라이브러리는 3.12에서 제거되어서, 현재 Python >= 3.12 에서는 설치가 정상적으로 되지 않습니다.
더이상
python-Wappanalyzer가 유지보수 되지 않아 pkg_resources 의존성을 제거한 포크버전을 추가했습니다. 제가 직접 만든거여서 리뷰 후에 메인테이너분이 직접 포크버전을 업로드하셔도 무방해 보입니다.또한 Docker에서 돌도록 dockerfile도 추가했는데 필요에 따라 README.md 도 간단하게 수정하면 좋아보입니다.