search

nwfsc-fram / research-catch-db

0 stars 2 forks source link

add warnings to log in page #49

Closed SaOgaz-NOAA closed 4 years ago

SaOgaz-NOAA commented 4 years ago

Login page is super sparse right now, doesn't give notification if log in doesn't work. Change password etc buttons aren't hooked up... leave those, implement them?

SaOgaz-NOAA commented 4 years ago

Hmm... this "works", but it also appears that bnauth(?) gives a total misnomer of an error when the pw is wrong. Should look into that at some point.

ghost commented 4 years ago

Indeed, bn-auth gets whatever error msg from AUA and passes it along. We could "translate it" perhaps

SaOgaz-NOAA commented 4 years ago

Hmmm. Would you expect this... bad pw gave me a "Error: User not authorized for BOATNET_OBSERVER"

ghost commented 4 years ago

That's straight from AUA, yes. (Generally auth failures should be generic and not specify why they failed)

SaOgaz-NOAA commented 4 years ago

Okay. For now I'll just change it to some generalized error. Not sure if we want to also do this for obs-web @sethgerou-noaa