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

Update main.py #3

Merged
merged 1 commit into from
Mar 2, 2021
Merged

Update main.py #3

merged 1 commit into from
Mar 2, 2021

Conversation

MilkClouds
Copy link
Contributor

2021-03-02 기준으로 작동하도록 소스코드 수정했습니다.
백준 로그인 쿠키 받아오는 부분은 Fiddler4같은 프로그램으로 읽어오시든 그냥 크롬 F12눌러서 Console창 열고 document.cookie 받아오시든 하면 됩니다. a=1;b=1; 식으로 나오는 걸 그대로 복사해서 붙여넣으시면 돼요.

다만 다운로드 속도가 지나치게 빠른데 이것은 서버에 무리를 줄 가능성이 있으므로 속도를 줄여서 쓸 필요성이 있어 보입니다. 또, C++98이나 Java 8과 같이 옛날 언어는 다운로드 할 때 오류가 나는 것을 확인했습니다. 아마도 extensions.py에서 언어 코드를 수정해줘야 하는 것으로 보입니다.

@isac322 isac322 self-assigned this Mar 2, 2021
@isac322 isac322 self-requested a review March 2, 2021 12:17
@isac322 isac322 added the bug label Mar 2, 2021
Copy link
Owner

@isac322 isac322 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
BOJ사용안한지 오래됐는데 작업해주셔서 감사합니다.
조만간 CI 붙여서 기능테스트를 정기적으로 돌리고 관리도 해줘야겠군요

@isac322 isac322 merged commit c5d4a29 into isac322:master Mar 2, 2021
@MilkClouds
Copy link
Contributor Author

MilkClouds commented Mar 2, 2021

지금 다시 보니 C++98이나 Java 8에 문제가 있는건, 옛날에 쓰던 언어가 없어져서 그런 게 아니라
백준에서 지금까지도 사용하는 언어인데 extensions.py에 안 들어있어서 그런듯합니다.
extensions.py에 key로 몇몇 언어 추가해주면 되겠네요.

그리고 로그인 할 때 cookie 사용하도록 한 건 정확한 원인은 저도 파악하지 않았으나 그냥 id,pw로 로그인하는게 안되더라고요. 아마도 캡차 때문이라고 추측은 하지만 확실하지는 않습니다. 그래서 그냥 사용자가 직접 쿠키를 복사&붙여넣기해서 로그인이 필요 없도록 수정했습니다.

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

Successfully merging this pull request may close these issues.

2 participants