PaulC91 / shinyauthr

R package with shiny authentication modules
https://paulc91.github.io/shinyauthr/
Other
427 stars 81 forks source link

Bug fix for apps not using cookie authentication feature #45

Closed michael-dewar closed 3 years ago

michael-dewar commented 3 years ago

This fixes a bug that I just found. If an app does not use the cookie authentication, then shinyauthr uses a default cookie getter function. Unfortunately this did not respect the app's choice of user_col and would crash unless the app happened to use user_col=user. I added an example app in inst/shiny-examples/broken/app.R that crashes under the old version (which was just merged) but works correctly with the changes contained in this PR.

Sorry about this.

PaulC91 commented 3 years ago

Thanks Michael!