search

Tychobra / polished

Authentication and Administration for Shiny apps
https://polished.tech
Other
234 stars 36 forks source link

Sentry option #150

Closed phoward38 closed 2 years ago

phoward38 commented 2 years ago

https://github.com/Tychobra/polished/blob/8d92f9cc21875f6c5636dc60960065f9cb5af5a8/R/secure_ui.R#L124

Shouldn't this be sentry_dsn <- getOption("polished")$sentry_dsn instead?

merlinoa commented 2 years ago

Yes updated thanks. I think it actually worked with just "sentry" due to R's partial matching, but certainly better to use the correct option name.