sureshanaparti commented on code in PR #11567:
URL: https://github.com/apache/cloudstack/pull/11567#discussion_r2429740973
##########
server/src/main/java/org/apache/cloudstack/backup/BackupManagerImpl.java:
##########
@@ -980,7 +980,7 @@ public BackupProvider getBackupProvider(final String name) {
@Override
public List<Class<?>> getCommands() {
final List<Class<?>> cmdList = new ArrayList<Class<?>>();
- if (!BackupFrameworkEnabled.value()) {
+ if (!BackupFrameworkEnabled.value() &&
!BackupFrameworkEnabled.hasValueInScope(Boolean.TRUE.toString())) {
Review Comment:
@shwstppr it doesn't restrict API access. it allows API access if it is
enabled in global or in any zone (not mandatory to enable in global scope).
previously, enabling at global level is mandatory for API access.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
