search

r4ds / shinyslack

Integrate Slack and Shiny
Other
15 stars 3 forks source link

Set Cookie via HTTP #34

Closed jonthegeek closed 1 year ago

jonthegeek commented 1 year ago

An http_only cookie would work better here. We can SET it with cookies::set_cookie_response(). If we READ it in the request and use that to set the environment variable for {slackteams}, is that enough?

jonthegeek commented 1 year ago

Instead of (or maybe in addition to?) looking in input for the cookie, look in session$request. This goes in check_login() at least, maybe other places?

As far as I can find/remember, it's JUST there. Would pass it session instead of input (which actually we can infer without it being passed directly, like in shiny::moduleServer().