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

관리자 페이지의 "관리자 ip 대역" #443

Closed
twocad opened this issue Feb 6, 2014 · 4 comments
Closed

관리자 페이지의 "관리자 ip 대역" #443

twocad opened this issue Feb 6, 2014 · 4 comments

Comments

@twocad
Copy link

twocad commented Feb 6, 2014

이번에 업데이트를 하고 나서보니
ip 대역에 등록이 되어있지 않은 ip에서는 관리자 페이지 접근이 불가하게 되었더군요.

같은 관리자라도 ip가 전혀 다른 곳에서의 접근도 많이 있을텐데
그런 부분에 대한 고려가 되어있지 않은 듯 해서 글 올립니다.

관리자 ip 대역을 풀어놓거나 잠그거나 하는 설정부분도 있었으면 좋겠습니다.

@akasima akasima self-assigned this Feb 7, 2014
@bjrambo
Copy link
Contributor

bjrambo commented Feb 7, 2014

광역 대역으로 설정도 가능합니다.
192.168.0.1-192.168.0.254 이런형태로 설정하면 되는걸로 알고 있습니다.
그리고.. 해당 공란을 비우게 되면 모든 관리자 계정으로 접속하는 사람들이 접속이 가능하지 않던가요?
아무튼 제가 테스트 해본결과로는 그 설정페이지에서 공란으로 비워두면 됩니다.

@twocad
Copy link
Author

twocad commented Feb 7, 2014

qw5414 / 제가 금방 다시 테스트 했는데 공란으로 하고 저장을 누르면
다른 아이피에서도 접속이 되더군요.
문제는 공란으로 했을때 공란으로 뜨지 않고 다시 아이피가 들어간 채로 뜹니다.
물론 그 상태에서 고급 탭을 저장하지 않고 사용한다면 계속 접속은 되겠지만
다른 설정을 변경할 때는 다시 아이피가 저장이 되어버리는 단점이 있네요.
다시 말해서 다른 아이피로 접속해서 저장을 해버리면 또 다른 곳에서는 접속이 금지되는셈이죠.
공란으로 비우고 저장을 하고나면 그 담부터는 공란으로 떠야 될것 같습니다.

@bjrambo
Copy link
Contributor

bjrambo commented Feb 7, 2014

@twocad 저도 그 부분은 고쳐야 한다고 생각합니다 :)

@ghost
Copy link

ghost commented Feb 7, 2014

@twocad 저의 실책입니다.
원래는 @khongchi 의 코드에서는 textarea에 title 속성으로만 처리되어 있었는데..
현재 접속한 실제 IP를 반드시 포함하도록 처리하다가 textarea에 값이 들어간 채로 저장되었네요. ㅠ

@ghost ghost closed this as completed Feb 7, 2014
@ghost ghost reopened this Feb 7, 2014
@ghost ghost assigned ghost and unassigned akasima Feb 7, 2014
@ghost ghost added the type/bug label Feb 7, 2014
@ghost ghost added this to the 1.7.5 milestone Feb 7, 2014
@ghost ghost added the priority/high label Feb 7, 2014
@ghost ghost closed this as completed Feb 8, 2014
@akasima akasima modified the milestones: 1.7.4.1, 1.7.5 Mar 13, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants