Skip to content

Commit

Permalink
disable LDAP since it interferes with update
Browse files Browse the repository at this point in the history
Fix #2208
  • Loading branch information
enoch85 authored Jan 30, 2022
1 parent 0128852 commit dceb1e6
Showing 1 changed file with 14 additions and 0 deletions.
14 changes: 14 additions & 0 deletions nextcloud_update.sh
Original file line number Diff line number Diff line change
Expand Up @@ -956,6 +956,13 @@ Please check in $BACKUP if config/ folder exist."
exit 1
fi

# Disable LDAP for now since it bugs out on NC 23.0.1
## TODO; remove in NC 23.0.2
if is_app_installed user_ldap
then
nextcloud_occ app:disable user_ldap

This comment has been minimized.

Copy link
@szaimen

szaimen Jan 30, 2022

Collaborator

What if the app was disabled before?

fi

if [ -d $BACKUP/apps/ ]
then
print_text_in_color "$ICyan" "$BACKUP/apps/ exists"
Expand Down Expand Up @@ -1007,6 +1014,13 @@ Please check in $BACKUP if the folders exist."
exit 1
fi

# Enable LDAP again
## TODO; remove in NC 23.0.2
if is_app_installed user_ldap
then
nextcloud_occ app:enable user_ldap

This comment has been minimized.

Copy link
@szaimen

szaimen Jan 30, 2022

Collaborator

What if the app wasn't enabled before?

fi

# Update Bitwarden
if is_docker_running
then
Expand Down

1 comment on commit dceb1e6

@enoch85
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@szaimen Fixed, see PR.

Please sign in to comment.