Skip to content

feat: add support for multiple user environment types and update rela… #2394

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

TangoBeeAkto
Copy link
Contributor

…ted functionality

…ted functionality

# Conflicts:
#	libs/dao/src/main/java/com/akto/dto/BackwardCompatibility.java
@TangoBeeAkto TangoBeeAkto force-pushed the feature/multi_tags_support_for_apicollections branch from ed21c86 to 41d8106 Compare May 27, 2025 10:36
@@ -142,25 +148,31 @@ public void setUrls(Set<String> urls) {
this.urls = urls;
}

public String getEnvType(){
public List<CollectionTags> getEnvType(){
Copy link
Contributor

Choose a reason for hiding this comment

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

For existing data tagsList will be null, but userSetEnv might be non null.

For users which has userSetEnv != null , we need to fill this value in tagsList right ?

@@ -163,6 +166,7 @@ public BackwardCompatibility(int id, int dropFilterSampleData, int resetSingleTy
this.changeOperatorConditionInCDT = changeOperatorConditionInCDT;
this.cleanupRbacEntries = cleanupRbacEntries;
this.fillLastTestedField = fillLastTestedField;
this.multipleUserEnvTypesSupport = multipleUserEnvTypesSupport;
Copy link
Contributor

Choose a reason for hiding this comment

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

what is the use of this ? we wanted to skip the cron right ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants