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.

Sign up for GitHub

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

Jump to bottom

ログインの問題の原因究明を容易にする #135

Closed
coord-e opened this issue Aug 27, 2018 · 10 comments
Closed

ログインの問題の原因究明を容易にする #135

coord-e opened this issue Aug 27, 2018 · 10 comments
Labels
after first release to do after the first release enhancement New feature or request small which is small, easy to review/fix

Comments

@coord-e
Copy link
Contributor

coord-e commented Aug 27, 2018

Step 1: 目的

  • ログインできない時原因究明困る

Step 2: 概要

  • production以外のconfigの時に、ログイン失敗についてログを吐く
@coord-e coord-e added enhancement New feature or request small which is small, easy to review/fix labels Aug 27, 2018
@Cj-bc
Copy link
Member

Cj-bc commented Aug 29, 2018

これでログ吐く必要があるのは以下の通りで大丈夫ですか？？

  1. reCAPTCHA認証が失敗した
  2. Secret_idが不正だった

@coord-e

@coord-e
Copy link
Contributor Author

coord-e commented Aug 29, 2018

はい

@coord-e coord-e changed the title ログインの原因究明を容易にする ログインの問題の原因究明を容易にする Aug 29, 2018
@Cj-bc
Copy link
Member

Cj-bc commented Aug 29, 2018

了解です

@Cj-bc Cj-bc assigned Cj-bc and unassigned Cj-bc Sep 6, 2018
@Cj-bc
Copy link
Member

Cj-bc commented Sep 8, 2018

ちょっと外れますね、pythonのロガー周りが思ったよりも複雑だった...

@coord-e
Copy link
Contributor Author

coord-e commented Sep 11, 2018

これ本番までにできます？

@Cj-bc
Copy link
Member

Cj-bc commented Sep 11, 2018

えと、ごめん #183回りに時間食われるので無理だと思う…
少なくとも私は無理かと

@Cj-bc
Copy link
Member

Cj-bc commented Sep 14, 2018

shino君が #183 やってくれてるのでこれやります
ログ吐くものはとりあえず上記二つ、レベルは...どうしよう

@Cj-bc
Copy link
Member

Cj-bc commented Sep 14, 2018 •
edited
Loading

この記事によると、DEBUG=Falseの際に出力されるログはerror以上らしいのでerrorにするしかない...?
ごめんなさいissueをきちんと読んでなかった...production以外ね了解

@Cj-bc Cj-bc added low priority This issue or pull request can be postponed after first release to do after the first release labels Sep 14, 2018
@Cj-bc
Copy link
Member

Cj-bc commented Sep 14, 2018

@coord-e と話して、作展に間に合わせなくていいとのことになったのでタグ付け

@Cj-bc Cj-bc removed the low priority This issue or pull request can be postponed label Sep 14, 2018
@shino16
Copy link
Member

shino16 commented Feb 6, 2020

問題ないので閉じます

@shino16 shino16 closed this as completed Feb 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
after first release to do after the first release enhancement New feature or request small which is small, easy to review/fix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@coord-e @Cj-bc @shino16