Skip to content

🔖 :: 1.1.5 릴리스 (develop → main) - #258

Merged
aiden30015 merged 9 commits into
mainfrom
develop
Aug 26, 2026
Merged

🔖 :: 1.1.5 릴리스 (develop → main)#258
aiden30015 merged 9 commits into
mainfrom
develop

Conversation

@aiden30015

Copy link
Copy Markdown
Contributor

💡 개요

1.1.5 릴리스 PR. develop 에 모인 변경사항을 main 으로 올린다.

🔗 관련 이슈

📃 작업내용

🔍 테스트 방법

Android 에뮬레이터(Pixel 9, API 36) + 프로덕션 서버로 실계정 로그인까지 확인 완료.

  1. DataGSM으로 로그인 탭 → Chrome Custom Tab 으로 oauth.datagsm.kr 로그인 페이지 열림
  2. 로그인 → Custom Tab 자동 종료, 앱으로 복귀
  3. POST /api/v2/auth/login → 200, 토큰 저장
  4. 후속 인증 API(users/my, reservations/active/room, machines/status) 전부 200, 홈 화면 정상 진입

🙋‍♂️ 질문사항

  • 인증 코드가 커스텀 스킴(com.washer.v2://auth/callback)으로 돌아오는 구조라, 같은 스킴을 등록한 앱이 코드를 가로챌 수 있다. 확인해보니 DataGSM OAuth 가 code_challenge 를 무시하고(PKCE 미지원) redirect_uri 도 정확일치 화이트리스트라, 앱 단독으로는 막을 수 없다. hellogsm 쪽에 PKCE 지원 또는 https://washer.com/auth/callback 등록을 요청하는 별도 이슈가 필요하다.
  • 브라우저 취소와 실제 실패를 모두 무음 처리 중이라, 브라우저가 열리지 않는 상황에선 버튼이 먹통으로 보인다. PlatformException.code == 'CANCELED' 만 무시하도록 후속 개선 여지가 있다.

✅ 체크리스트

  • 코드가 정상적으로 컴파일되고 실행되는지 확인했습니다.
  • 불필요한 코드가 없는지 확인했습니다.
  • 코드 스타일 가이드를 준수했습니다.
  • 기존 기능이 정상적으로 작동하는지 확인했습니다.

aiden30015 and others added 9 commits August 25, 2026 15:55
- flutter_web_auth_2 로 Chrome Custom Tabs / ASWebAuthenticationSession 인증 후 콜백 캡처
- AuthWebViewScreen 및 /auth/webview 라우트 제거, webview_flutter 의존성 제거
- Android 콜백 intent-filter 를 CallbackActivity 로 이동

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
취소 커밋과 상태조회 API 응답 사이 타이밍 지연으로, 아무도 점유하지
않은 기기가 사용중으로 보여 재예약이 막히는 경우가 있어 짧은
재확인(500ms) 로직을 추가했다.

Close #256
🔀 :: (#252) - 공통 에러 메시지 처리 개선
…-user-agent

🔀 :: (#253) - 로그인 방식을 WebView 에서 시스템 브라우저(user-agent)로 변경
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
🔀 :: (#256) - 취소 직후 재예약 시 AlreadyReservedException 오탐 완화
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@aiden30015
aiden30015 merged commit bc592b0 into main Aug 26, 2026
3 checks passed
@aiden30015
aiden30015 deleted the develop branch August 26, 2026 07:47
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.

3 participants