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

開発用のプロキシサーバを追加 #600

Closed
wants to merge 1 commit into from
Closed

Conversation

HosokawaR
Copy link
Member

@HosokawaR HosokawaR commented Nov 11, 2022

現状この方法だと認証ができないため Draft
自分が開発をするときは本番環境で取得した twinte_session Cookie を手動でコピーしていた。

背景

#601 での Service Worker の開発に必要になったため追加した。
現行の Readme に書いてある、拡張機能を用いた CORS エラーの回避方法はブラウザのページスクリプト内でのみ使えるので、それとは別のところで動いている Service Worker では使えない。

また拡張機能で CORS エラーを回避する方法は、拡張機能が使えるブラウザのみでしか使えない方法であり、また拡張機能の仕様変更に左右されてしまう。

さらに上記の拡張機能はブラウザのリクエストに介入して CORS ヘッダーを改竄しており、本番環境とことなる CORS ヘッダーとなるので、動作環境などで本番環境と差異がでる可能性がある。

したがって Service Worker の開発に限らず、拡張機能に依存しないで、本番環境となるべく近い動作する CORS エラーを回避できる手法がほしかった。

変更点

ローカルで立てるプロキシサーバを追加した。

@vercel
Copy link

vercel bot commented Nov 11, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
twinte-front ✅ Ready (Inspect) Visit Preview Nov 11, 2022 at 0:58AM (UTC)

@HosokawaR
Copy link
Member Author

再開の機運がないのでこの PR は一旦閉じる

@HosokawaR HosokawaR closed this May 2, 2023
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.

1 participant