How we found and fixed a rare race condition in our session handling

EngineeringSecurityHow we found and fixed a rare race condition in our session handlingOn March 8, we shared that, out of an abundance of caution, we logged all users out of GitHub.com due to a rare security vulnerability. We believe that transparency is… On March 8, we shared that, out of an abundance of caution, we logged all users out of GitHub.com due to a rare security vulnerability. We belie

[criticabug]

これもセッションスワップ バグかな。セキュアプログラミング学んでスレッドセーフかどうか気を付けてコーディングして負荷テストして…

[tkmkg8m]

Rackの env オブジェクトがリクエストまたいで使い回されてて、本来クリアされるはずがレースコンディションにより変に更新されてしまったと。これはセッションリセットするね……。人類にスレッドは早すぎたんや。

[ktsujister]

“ a collaborative team of sub”

[uzulla]

Unicornでもこういう事あるんだな。

[onigra]

“We believe that transparency is key in earning and keeping the trust of our users and want to share more about this bug.”