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

회원 캐시를 비우면 관련된 전역변수도 비우도록 변경 #2372

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

kijin
Copy link
Contributor

@kijin kijin commented Jan 28, 2019

https://xetown.com/questions/1144200 에서 제보된 문제의 원인이 아닐까 의심되는 부분입니다.

memberController->_clearMemberCache()를 호출하면 오브젝트 캐시는 비우는데 여기저기 전역변수로 저장해 놓은 캐시는 그대로 두고 있어서, 회원정보를 변경하고 동일한 요청 내에서 프로필 이미지, 서명 등을 불러오려고 하면 오작동할 가능성이 있습니다.

회원 캐시를 비우면 memberModel에서 사용하는 전역변수 캐시도 모두 비우도록 수정해 보았습니다.

@ghost ghost self-assigned this Jan 29, 2019
@ghost ghost added this to the 1.11-next milestone Apr 29, 2019
@ghost ghost removed their assignment Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant