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

fix(frontend): Chromeのパスワードマネージャーにあらゆる入力欄がパスワード扱いをさせる問題を回避 #13141

Closed
wants to merge 5 commits into from

Conversation

kakkokari-gtyih
Copy link
Contributor

@kakkokari-gtyih kakkokari-gtyih commented Feb 2, 2024

What

  1. inputタグにタイプ別に違うクラスを割り当て、ヒューリスティックアルゴリズムが違うものであることを区別できるように
  2. pwのフィールドだと判定される要素を一つでも減らせる

(cherry picked from commit 6cd4ea5)

Why

ioではこの問題によりかなり操作性が損なわれていたので、他のサーバーでもこの現象が再現されるならマージしてもいいかなと思った(逆にio以外で発生してないのならマージしなくても問題ないかも)

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

u1-liquid and others added 2 commits February 2, 2024 15:00
…#392)

1. inputタグにタイプ別に違うクラスを割り当て、ヒューリスティックアルゴリズムが違うものであることを区別できるように
2. `pw`のフィールドだと判定される要素を一つでも減らせる

(cherry picked from commit 6cd4ea5)
@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Feb 2, 2024
Copy link

codecov bot commented Feb 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c38f5ee) 64.31% compared to head (88b5a17) 77.52%.
Report is 27 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #13141       +/-   ##
============================================
+ Coverage    64.31%   77.52%   +13.20%     
============================================
  Files          977      183      -794     
  Lines       109255    24667    -84588     
  Branches      5581      462     -5119     
============================================
- Hits         70266    19122    -51144     
+ Misses       37558     5538    -32020     
+ Partials      1431        7     -1424     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

なんだこれは

@tamaina
Copy link
Contributor

tamaina commented Feb 2, 2024

Chromeのバグじゃないんか

(コメントがないとこのclassがなんのためのものなのかわからなくなりそうな気がする

@kakkokari-gtyih
Copy link
Contributor Author

(コメントがないとこのclassがなんのためのものなのかわからなくなりそうな気がする

done

@syuilo
Copy link
Member

syuilo commented Feb 2, 2024

効果ないかも

@u1-liquid
Copy link
Sponsor Contributor

chromeのサイトの表示問題の報告にreport飛ばしまくるしかないかもしれない

@kakkokari-gtyih
Copy link
Contributor Author

chromeのサイトの表示問題の報告にreport飛ばしまくるしかないかもしれない

というわけで

@kakkokari-gtyih
Copy link
Contributor Author

これしてたほうがマシな気がする(なんかまたパスワード欄に誤検知されるようになった)

@kakkokari-gtyih
Copy link
Contributor Author

嘘(関係なかった)

@tamaina
Copy link
Contributor

tamaina commented Feb 10, 2024

ふーむ

Screenshot_20240210-151420.png

@tamaina
Copy link
Contributor

tamaina commented Feb 10, 2024

あっこれPRなのか

(なお直らない模様

@tamaina
Copy link
Contributor

tamaina commented Feb 17, 2024

issueどこ?ない?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants