이메일 주소 변경 시 내부 서버 오류 발생

[Richie (Richie Rich)]

Discourse 3.4.0.beta4-dev (5bc7371192).

사용자는 이전에 사용했던 이메일 주소를 다시 변경할 수 없습니다.

재현 방법:

1. 계정 이메일 주소를 + 기호가 포함된 것으로 변경합니다. 예: your+name@example.com

(여기서 + 기호가 중요한 역할을 할 수도 있고, 단순한 혼동일 수도 있습니다)

2. 이메일 안내에 따라 변경 사항을 확인합니다

3. 이메일 주소를 이전 값으로 다시 변경합니다. 예: yourname@example.com

4. 이메일 안내에 따라 변경 사항을 확인합니다

5. 프론트엔드에서 오류가 발생합니다:

그 후 사용자는 다음과 같은 상태에 갇힙니다:

---

/logs 페이지의 오류:

lib/email_updater.rb:136:in `update_user_email'

lib/email_updater.rb:126:in `block in confirm'

activerecord (7.2.2.1) lib/active_record/connection_adapters/abstract/transaction.rb:616:in `block in within_new_transaction'

activesupport (7.2.2.1) lib/active_support/concurrency/null_lock.rb:9:in `synchronize'

activerecord (7.2.2.1) lib/active_record/connection_adapters/abstract/transaction.rb:613:in `within_new_transaction'

activerecord (7.2.2.1) lib/active_record/connection_adapters/abstract/database_statements.rb:361:in `transaction'

activerecord (7.2.2.1) lib/active_record/transactions.rb:234:in `block in transaction'

activerecord (7.2.2.1) lib/active_record/connection_adapters/abstract/connection_pool.rb:421:in `with_connection'

activerecord (7.2.2.1) lib/active_record/connection_handling.rb:296:in `with_connection'

activerecord (7.2.2.1) lib/active_record/transactions.rb:233:in `transaction'

lib/email_updater.rb:87:in `confirm'

app/controllers/users_email_controller.rb:64:in `confirm_new_email'

actionpack (7.2.2.1) lib/action_controller/metal/basic_implicit_render.rb:8:in `send_action'

actionpack (7.2.2.1) lib/abstract_controller/base.rb:226:in `process_action'

actionpack (7.2.2.1) lib/action_controller/metal/rendering.rb:193:in `process_action'

actionpack (7.2.2.1) lib/abstract_controller/callbacks.rb:261:in `block in process_action'

activesupport (7.2.2.1) lib/active_support/callbacks.rb:121:in `block in run_callbacks'

app/controllers/application_controller.rb:424:in `block in with_resolved_locale'

i18n (1.14.7) lib/i18n.rb:353:in `with_locale'

app/controllers/application_controller.rb:424:in `with_resolved_locale'

activesupport (7.2.2.1) lib/active_support/callbacks.rb:130:in `block in run_callbacks'

activesupport (7.2.2.1) lib/active_support/callbacks.rb:141:in `run_callbacks'

actionpack (7.2.2.1) lib/abstract_controller/callbacks.rb:260:in `process_action'

actionpack (7.2.2.1) lib/action_controller/metal/rescue.rb:27:in `process_action'

actionpack (7.2.2.1) lib/action_controller/metal/instrumentation.rb:77:in `block in process_action'

activesupport (7.2.2.1) lib/active_support/notifications.rb:210:in `block in instrument'

activesupport (7.2.2.1) lib/active_support/notifications/instrumenter.rb:58:in `instrument'

activesupport (7.2.2.1) lib/active_support/notifications.rb:210:in `instrument'

actionpack (7.2.2.1) lib/action_controller/metal/instrumentation.rb:76:in `process_action'

actionpack (7.2.2.1) lib/action_controller/metal/params_wrapper.rb:259:in `process_action'

activerecord (7.2.2.1) lib/active_record/railties/controller_runtime.rb:39:in `process_action'

actionpack (7.2.2.1) lib/abstract_controller/base.rb:163:in `process'

actionview (7.2.2.1) lib/action_view/rendering.rb:40:in `process'

actionpack (7.2.2.1) lib/action_controller/metal.rb:252:in `dispatch'

actionpack (7.2.2.1) lib/action_controller/metal.rb:335:in `dispatch'

actionpack (7.2.2.1) lib/action_dispatch/routing/route_set.rb:67:in `dispatch'

actionpack (7.2.2.1) lib/action_dispatch/routing/route_set.rb:50:in `serve'

actionpack (7.2.2.1) lib/action_dispatch/journey/router.rb:53:in `block in serve'

actionpack (7.2.2.1) lib/action_dispatch/journey/router.rb:133:in `block in find_routes'

actionpack (7.2.2.1) lib/action_dispatch/journey/router.rb:126:in `each'

actionpack (7.2.2.1) lib/action_dispatch/journey/router.rb:126:in `find_routes'

actionpack (7.2.2.1) lib/action_dispatch/journey/router.rb:34:in `serve'

actionpack (7.2.2.1) lib/action_dispatch/routing/route_set.rb:896:in `call'

lib/middleware/omniauth_bypass_middleware.rb:64:in `call'

rack (2.2.10) lib/rack/tempfile_reaper.rb:15:in `call'

rack (2.2.10) lib/rack/conditional_get.rb:40:in `call'

rack (2.2.10) lib/rack/head.rb:12:in `call'

actionpack (7.2.2.1) lib/action_dispatch/http/permissions_policy.rb:38:in `call'

lib/content_security_policy/middleware.rb:12:in `call'

lib/middleware/anonymous_cache.rb:403:in `call'

lib/middleware/csp_script_nonce_injector.rb:12:in `call'

config/initializers/008-rack-cors.rb:14:in `call'

rack (2.2.10) lib/rack/session/abstract/id.rb:266:in `context'

rack (2.2.10) lib/rack/session/abstract/id.rb:260:in `call'

actionpack (7.2.2.1) lib/action_dispatch/middleware/cookies.rb:704:in `call'

actionpack (7.2.2.1) lib/action_dispatch/middleware/callbacks.rb:31:in `block in call'

activesupport (7.2.2.1) lib/active_support/callbacks.rb:101:in `run_callbacks'

actionpack (7.2.2.1) lib/action_dispatch/middleware/callbacks.rb:30:in `call'

actionpack (7.2.2.1) lib/action_dispatch/middleware/debug_exceptions.rb:31:in `call'

actionpack (7.2.2.1) lib/action_dispatch/middleware/show_exceptions.rb:32:in `call'

logster (2.20.0) lib/logster/middleware/reporter.rb:40:in `call'

railties (7.2.2.1) lib/rails/rack/logger.rb:41:in `call_app'

railties (7.2.2.1) lib/rails/rack/logger.rb:29:in `call'

config/initializers/100-quiet_logger.rb:20:in `call'

config/initializers/100-silence_logger.rb:29:in `call'

actionpack (7.2.2.1) lib/action_dispatch/middleware/request_id.rb:33:in `call'

lib/middleware/enforce_hostname.rb:24:in `call'

rack (2.2.10) lib/rack/method_override.rb:24:in `call'

actionpack (7.2.2.1) lib/action_dispatch/middleware/executor.rb:16:in `call'

rack (2.2.10) lib/rack/sendfile.rb:110:in `call'

lib/middleware/processing_request.rb:12:in `call'

message_bus (4.3.8) lib/message_bus/rack/middleware.rb:60:in `call'

lib/middleware/request_tracker.rb:385:in `call'

actionpack (7.2.2.1) lib/action_dispatch/middleware/remote_ip.rb:96:in `call'

railties (7.2.2.1) lib/rails/engine.rb:535:in `call'

railties (7.2.2.1) lib/rails/railtie.rb:226:in `public_send'

railties (7.2.2.1) lib/rails/railtie.rb:226:in `method_missing'

rack (2.2.10) lib/rack/urlmap.rb:74:in `block in call'

rack (2.2.10) lib/rack/urlmap.rb:58:in `each'

rack (2.2.10) lib/rack/urlmap.rb:58:in `call'

unicorn (6.1.0) lib/unicorn/http_server.rb:634:in `process_client'

unicorn (6.1.0) lib/unicorn/http_server.rb:739:in `worker_loop'

unicorn (6.1.0) lib/unicorn/http_server.rb:547:in `spawn_missing_workers'

unicorn (6.1.0) lib/unicorn/http_server.rb:143:in `start'

unicorn (6.1.0) bin/unicorn:128:in `<top (required)>'

vendor/bundle/ruby/3.3.0/bin/unicorn:25:in `load'

vendor/bundle/ruby/3.3.0/bin/unicorn:25:in `<main>'

[Canapin (Coin-coin le Canapin)]

@Richie

(플러스 기호가 여기서 핵심일 수도 있고, 단순한 오해일 수도 있습니다)

기본적으로 이메일 정규화 활성화

배경 과거에 Discourse는 이메일 정규화(normalization)를 비활성화된 상태로 배포되었습니다. 그 결과, 플러스 어드레싱(plus addressing)을 사용하여 동일한 정규화(canonical) 이메일로 여러 계정을 만들 수 있었습니다. 예를 들어, bob@discourse.org에 연결된 계정이 있는 사용자는 bob+again@discourse.org에 연결된 또 다른 계정을 만들 수 있습니다. Discourse에는 이를 허용하거나 허용하지 않도록 전환할 수 있는 normalize_emails 사이트 설정이 있습니다. 이 설정이 켜져 있으면, 위의 두 이메일을 고유 계정 검증 목적에서 동일한 것으로 간주하므로 Bob은 두 번째 계정을 만들 수 없습니다. 왜 변경하나요? 최근 스팸 가입이 크게 증가했으며, 플러스 어드레싱을 사용하여 단일 이메일로부터 수백 건에서 수천 건에 달하는 가입이 발생하는 사이트도 있었습니다. 당연히 이런 일이 발생하면 상당히 불쾌한 일이 …

normalize_emails 설정을 확인해 보셨나요? 여기에 버그가 없다고 말하려는 것은 아닙니다

[Richie (Richie Rich)]

아니요, 그 설정을 확인해본 적이 없습니다. 그런 설정이 있는지 몰랐거든요. 감사합니다 @Canapin

이것이 문제의 원인이라면 Internal Server Error 버그를 잡아 사용자가 이 이메일 주소를 사용할 수 없는 이유를 알려줄 수 있도록 처리해야 합니다.

[sam (Sam Saffron)]

앞으로 4주 안에 누군가가 이 문제를 살펴볼 예정이니, “internal server error” 대신 사용자에게 명확한 경고를 표시해야 합니다.

[Richie (Richie Rich)]

@sam 감사합니다

[davidb]

이 문제는 다음 커밋으로 수정되었습니다:

FIX: ensure old email set correctly when updating email (#32309)

확인되지 않은 이메일 변경 사항이 있는 경우, 사용자 설정 페이지(예: /my/preferences/account)에서 확인되지 않은 이메일 옆의 렌치 아이콘을 클릭한 뒤 이메일 삭제를 눌러 먼저 삭제하는 것을 권장합니다.

이후 이메일 변경 절차를 다시 시작하면 이번에는 정상적으로 작동할 것으로 기대됩니다.