nextcloud / user_external

👥 External user authentication methods like IMAP, SMB and FTP
https://apps.nextcloud.com/apps/user_external
108 stars 64 forks source link

Rename new imap backend OC_IMAP_Auth2 #72

Closed mmccarn closed 5 years ago

mmccarn commented 5 years ago

I'd like to propose that User_External use a different 'user_backend' for the new vs old imap -- especially since the back-ends are so different, with differing data and configuration requirements.

Option 1 [protect admins still using v0.5.0 during upgrades]:

Option 2 [protect admins already using v0.6.x during upgrades]:

Option 3 [disconnect user_external from IMAP Authentication to protect admins during upgrades]:

violoncelloCH commented 5 years ago

I can understand the idea behind that and it would certainly have been a nice idea to leave the admins time to migrate like that... however I think it's to late for that now; it doesn't make sense to revert everything again now... only option that could make sense would be option 2; however this would need database migrations as well to not loose existing shares and stuff (see #64)... unless there is a big demand for it, I don't think it makes sense to re-add the old imap as a legacy option... not last because of security concerns if users keep the legacy method "forever"...

So, sorry, but I tend to close this...