imapc: Remove auth_failed check from imapc_list_get_storage_name()

Reverts 9949dc3649ac7a00289f6bf0662c793bee3d4286. It doesn't seem to be
actually needed at least now, and it's breaking:

 * SELECT INBOX -> failure due to remote server being down
 * SELECT INBOX -> NO [CANNOT] Invalid mailbox name: Name is empty

diff --git a/src/lib-storage/index/imapc/imapc-list.c b/src/lib-storage/index/imapc/imapc-list.c
index 67d600d4f6857bd5bb52fbaaf95f76341743727e..1091d95ef7e81cb889529e97a320d531dc42f524 100644 (file)
--- a/src/lib-storage/index/imapc/imapc-list.c
+++ b/src/lib-storage/index/imapc/imapc-list.c
@@ -347,11 +347,6 @@ imapc_list_get_storage_name(struct mailbox_list *_list, const char *vname)
        const char *prefix = list->set->imapc_list_prefix;
        const char *storage_name;
 
-       /* check if authentication has failed, if it has, short circuit here
-          to avoid the error being clobbered by mailbox_list_get_hierarchy_sep */
-       if (list->client->auth_failed)
-               return "";
-
        storage_name = mailbox_list_default_get_storage_name(_list, vname);
        if (*prefix != '\0' && strcasecmp(storage_name, "INBOX") != 0) {
                storage_name = storage_name[0] == '\0' ? prefix :