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

外部公開アンケート対応 #618

Open
mazrean opened this issue Sep 21, 2021 · 9 comments
Open

外部公開アンケート対応 #618

mazrean opened this issue Sep 21, 2021 · 9 comments
Assignees
Labels
enhancement New feature or request ready
Projects
Milestone

Comments

@mazrean
Copy link
Member

mazrean commented Sep 21, 2021

外部公開のアンケートの作成をできるようにしたい。
とりあえず使用を決めるのをやる。

@mazrean mazrean self-assigned this Sep 21, 2021
@mazrean mazrean added this to To do in anke-to via automation Sep 21, 2021
@mazrean mazrean added the enhancement New feature or request label Sep 21, 2021
@mazrean mazrean moved this from To do to In progress in anke-to Sep 21, 2021
@mazrean
Copy link
Member Author

mazrean commented Sep 21, 2021

仕様策定のために必要なことの整理

やりたいこと

  • アンケートの設定によってtraP外の人でも回答できる・できないを切り替えられる
    • 回答ができるのは必須
    • その後の修正、etcができるようにするのかは要検討
  • 結果閲覧は一旦後回し

エンドポイントの権限

traP外の人に権限を与える必要のある

回答の確認・修正・削除を許容したいなら必要

問題点

回答の確認・修正・削除を許容する場合、traP外の人の識別が必要。
用途として新歓の体験会の感想とかの1回答えて修正できなくても問題ないものが想定されているので、
実装コストとかを考えると切るのが正しい気がしている。

やらないといけないこと

  • 認証のcaddy-trap-authからの切り離し
    • 今まではcaddy-trap-authでtraP外の人のアクセスを弾いていた
    • アンケートの設定に応じて弾く弾かないの分岐が必要になる
    • anke-to内での認証が必要
  • DBのrespondentsテーブルのtraP外ユーザー対応
    • 現在はユーザー=user_traqidだが、これが成り立たなくなるので
  • questionnaireに外部公開・非公開の設定追加

@mazrean
Copy link
Member Author

mazrean commented Sep 23, 2021

一旦、回答の確認・修正・削除は後回しにするので確定。

@mazrean
Copy link
Member Author

mazrean commented Sep 23, 2021

今まではcaddy-trap-authでtraP外の人のアクセスを弾いていた

は若干誤りだった。
現在、caddy-trap-authのデフォルトのhardに設定されているのでcaddy-trap-authでリクエストが弾かれているけど、
softの場合caddy-trap-authはtraP部員以外でもX-Showcase-Userを-で上書きするので、
https://github.com/traPtitech/anke-to/blob/master/router/middleware.go#L61-L64
で弾かれる。

ただ、エンドポイントに応じてtraP外の人を通すか通さないか切り替える必要があるので、認証の修正は必要。

@mazrean
Copy link
Member Author

mazrean commented Sep 28, 2021

そういえば外部公開するならurl予測されないようにランダムなクエリパラメーター設定とかするべき?

@mazrean
Copy link
Member Author

mazrean commented Sep 28, 2021

idがauto_incrementだから現状だと簡単に予測はできる

@mazrean
Copy link
Member Author

mazrean commented Sep 28, 2021

やるとしても後回しが正解かな

@mazrean
Copy link
Member Author

mazrean commented Apr 6, 2022

traP Collectionでの工大祭のアンケートで使いたいと言うのがあるので、工大祭までには欲しい

@mazrean mazrean added the v2 label May 8, 2022
@mazrean
Copy link
Member Author

mazrean commented May 8, 2022

v2のmdより

  • 今は完全に部内のみのアクセス
  • 利用できそうな場面
    • 新歓、p4k, GAME^3, 工大祭
  • どこを公開するか
    • 回答 (する) 画面
      • サイドバーとかは非表示にする必要がある
      • Google Forms みたいな画面になりそう
      • 回答後の遷移
        • 「ありがとうございました」みたいな一文を表示するとか

@MotoyaAsahina
Copy link
Member

  • 外部公開アンケートに部員として回答できるか
    • 別々のアンケートとして対応できる
    • 部員の情報は取らなくて良い

@FujishigeTemma FujishigeTemma added help wanted Extra attention is needed ready and removed help wanted Extra attention is needed labels May 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ready
Projects
anke-to
In progress
Development

No branches or pull requests

3 participants