Versioned profiles feature flag.

app/src/main/java/org/thoughtcrime/securesms/jobs/ProfileUploadJob.java

@@ -53,7 +53,7 @@ protected void onRun() throws Exception {
     String      avatarPath  = null;
 
     try (StreamDetails avatar = AvatarHelper.getSelfProfileAvatarStream(context)) {
-      if (FeatureFlags.VERSIONED_PROFILES) {
+      if (FeatureFlags.versionedProfiles()) {
         avatarPath = accountManager.setVersionedProfile(Recipient.self().getUuid().get(), profileKey, profileName.serialize(), avatar).orNull();
       } else {
         accountManager.setProfileName(profileKey, profileName.serialize());

app/src/main/java/org/thoughtcrime/securesms/jobs/RefreshOwnProfileJob.java

@@ -93,7 +93,7 @@ private void setProfileKeyCredential(@NonNull Recipient recipient,
   }
 
   private static SignalServiceProfile.RequestType getRequestType(@NonNull Recipient recipient) {
-    return FeatureFlags.VERSIONED_PROFILES && !recipient.hasProfileKeyCredential()
+    return FeatureFlags.versionedProfiles() && !recipient.hasProfileKeyCredential()
            ? SignalServiceProfile.RequestType.PROFILE_AND_CREDENTIAL
            : SignalServiceProfile.RequestType.PROFILE;
   }

app/src/main/java/org/thoughtcrime/securesms/jobs/RetrieveProfileJob.java

@@ -145,7 +145,7 @@ private void setProfileKeyCredential(@NonNull Recipient recipient,
   }
 
   private static SignalServiceProfile.RequestType getRequestType(@NonNull Recipient recipient) {
-    return FeatureFlags.VERSIONED_PROFILES && !recipient.hasProfileKeyCredential()
+    return FeatureFlags.versionedProfiles() && !recipient.hasProfileKeyCredential()
            ? SignalServiceProfile.RequestType.PROFILE_AND_CREDENTIAL
            : SignalServiceProfile.RequestType.PROFILE;
   }

app/src/main/java/org/thoughtcrime/securesms/jobs/RotateProfileKeyJob.java

@@ -57,7 +57,7 @@ public void onRun() throws Exception {
     recipientDatabase.setProfileKey(self.getId(), profileKey);
 
      try (StreamDetails avatarStream = AvatarHelper.getSelfProfileAvatarStream(context)) {
-      if (FeatureFlags.VERSIONED_PROFILES) {
+      if (FeatureFlags.versionedProfiles()) {
         accountManager.setVersionedProfile(self.getUuid().get(),
                                            profileKey,
                                            Recipient.self().getProfileName().serialize(),

app/src/main/java/org/thoughtcrime/securesms/util/FeatureFlags.java

@@ -11,6 +11,7 @@
 import org.json.JSONException;
 import org.json.JSONObject;
 import org.thoughtcrime.securesms.dependencies.ApplicationDependencies;
+import org.thoughtcrime.securesms.jobs.ProfileUploadJob;
 import org.thoughtcrime.securesms.jobs.RefreshAttributesJob;
 import org.thoughtcrime.securesms.jobs.RemoteConfigRefreshJob;
 import org.thoughtcrime.securesms.keyvalue.SignalStore;
@@ -59,8 +60,9 @@ public final class FeatureFlags {
   private static final String REMOTE_DELETE              = "android.remoteDelete";
   private static final String PROFILE_FOR_CALLING        = "android.profileForCalling";
   private static final String CALLING_PIP                = "android.callingPip";
-  private static final String NEW_GROUP_UI               = "android.newGroupUI";
   private static final String REACT_WITH_ANY_EMOJI       = "android.reactWithAnyEmoji";
+  private static final String NEW_GROUP_UI               = "android.newGroupUI";
+  private static final String VERSIONED_PROFILES         = "android.versionedProfiles";
   private static final String GROUPS_V2                  = "android.groupsv2";
   private static final String GROUPS_V2_CREATE           = "android.groupsv2.create";
 
@@ -81,7 +83,8 @@ public final class FeatureFlags {
       PROFILE_FOR_CALLING,
       CALLING_PIP,
       NEW_GROUP_UI,
-      REACT_WITH_ANY_EMOJI
+      REACT_WITH_ANY_EMOJI,
+      VERSIONED_PROFILES
   );
 
   /**
@@ -113,6 +116,7 @@ public final class FeatureFlags {
   private static final Set<String> STICKY = Sets.newHashSet(
       PINS_FOR_ALL_LEGACY,
       PINS_FOR_ALL,
+      VERSIONED_PROFILES,
       GROUPS_V2
   );
 
@@ -128,8 +132,9 @@ public final class FeatureFlags {
    * desired test state.
    */
   private static final Map<String, OnFlagChange> FLAG_CHANGE_LISTENERS = new HashMap<String, OnFlagChange>() {{
-    put(MESSAGE_REQUESTS, (change) -> SignalStore.setMessageRequestEnableTime(change == Change.ENABLED ? System.currentTimeMillis() : 0));
-    put(GROUPS_V2,        (change) -> ApplicationDependencies.getJobManager().add(new RefreshAttributesJob()));
+    put(MESSAGE_REQUESTS,   (change) -> SignalStore.setMessageRequestEnableTime(change == Change.ENABLED ? System.currentTimeMillis() : 0));
+    put(VERSIONED_PROFILES, (change) -> ApplicationDependencies.getJobManager().add(new ProfileUploadJob()));
+    put(GROUPS_V2,          (change) -> ApplicationDependencies.getJobManager().add(new RefreshAttributesJob()));
   }};
 
   private static final Map<String, Object> REMOTE_VALUES = new TreeMap<>();
@@ -260,9 +265,14 @@ public static boolean reactWithAnyEmoji() {
     return getBoolean(REACT_WITH_ANY_EMOJI, false);
   }
 
+  /** Read and write versioned profile information. */
+  public static boolean versionedProfiles() {
+    return getBoolean(VERSIONED_PROFILES, false);
+  }
+
   /** Groups v2 send and receive. */
   public static boolean groupsV2() {
-    return getBoolean(GROUPS_V2, false);
+    return versionedProfiles() && getBoolean(GROUPS_V2, false);
   }
 
   /** Groups v2 send and receive. */
@@ -504,7 +514,4 @@ interface OnFlagChange {
   enum Change {
     ENABLED, DISABLED, CHANGED, REMOVED
   }
-
-  /** Read and write versioned profile information. */
-  public static final boolean VERSIONED_PROFILES = org.whispersystems.signalservice.FeatureFlags.VERSIONED_PROFILES;
 }

libsignal/service/src/main/java/org/whispersystems/signalservice/FeatureFlags.java

@@ -6,6 +6,6 @@
  */
 public final class FeatureFlags {
 
-  /** Read and write versioned profile information. */
-  public static final boolean VERSIONED_PROFILES = false;
+  /** Prevent usage of non-versioned profile endpoints. */
+  public static final boolean DISALLOW_OLD_PROFILE_SETTING = false;
 }

libsignal/service/src/main/java/org/whispersystems/signalservice/api/SignalServiceAccountManager.java

@@ -648,7 +648,7 @@ public TurnServerInfo getTurnServerInfo() throws IOException {
   public void setProfileName(ProfileKey key, String name)
       throws IOException
   {
-    if (FeatureFlags.VERSIONED_PROFILES) {
+    if (FeatureFlags.DISALLOW_OLD_PROFILE_SETTING) {
       throw new AssertionError();
     }
 
@@ -662,7 +662,7 @@ public void setProfileName(ProfileKey key, String name)
   public Optional<String> setProfileAvatar(ProfileKey key, StreamDetails avatar)
       throws IOException
   {
-    if (FeatureFlags.VERSIONED_PROFILES) {
+    if (FeatureFlags.DISALLOW_OLD_PROFILE_SETTING) {
       throw new AssertionError();
     }
 
@@ -684,10 +684,6 @@ public Optional<String> setProfileAvatar(ProfileKey key, StreamDetails avatar)
   public Optional<String> setVersionedProfile(UUID uuid, ProfileKey profileKey, String name, StreamDetails avatar)
       throws IOException
   {
-    if (!FeatureFlags.VERSIONED_PROFILES) {
-      throw new AssertionError();
-    }
-
     if (name == null) name = "";
 
     byte[]            ciphertextName    = new ProfileCipher(profileKey).encryptName(name.getBytes(StandardCharsets.UTF_8), ProfileCipher.NAME_PADDED_LENGTH);

libsignal/service/src/main/java/org/whispersystems/signalservice/api/SignalServiceMessagePipe.java

@@ -187,7 +187,7 @@ public ProfileAndCredential getProfile(SignalServiceAddress address,
                                                                        .setVerb("GET")
                                                                        .addAllHeaders(headers);
 
-      if (FeatureFlags.VERSIONED_PROFILES && requestType == SignalServiceProfile.RequestType.PROFILE_AND_CREDENTIAL && uuid.isPresent() && profileKey.isPresent()) {
+      if (requestType == SignalServiceProfile.RequestType.PROFILE_AND_CREDENTIAL && uuid.isPresent() && profileKey.isPresent()) {
         UUID                               target               = uuid.get();
         ProfileKeyVersion                  profileKeyIdentifier = profileKey.get().getProfileKeyVersion(target);
                                            requestContext       = clientZkProfile.createProfileKeyCredentialRequestContext(random, target, profileKey.get());

libsignal/service/src/main/java/org/whispersystems/signalservice/api/SignalServiceMessageReceiver.java

@@ -130,7 +130,7 @@ public ProfileAndCredential retrieveProfile(SignalServiceAddress address,
   {
     Optional<UUID> uuid = address.getUuid();
 
-    if (FeatureFlags.VERSIONED_PROFILES && requestType == SignalServiceProfile.RequestType.PROFILE_AND_CREDENTIAL && uuid.isPresent() && profileKey.isPresent()) {
+    if (requestType == SignalServiceProfile.RequestType.PROFILE_AND_CREDENTIAL && uuid.isPresent() && profileKey.isPresent()) {
       return socket.retrieveProfile(uuid.get(), profileKey.get(), unidentifiedAccess);
     } else {
       return new ProfileAndCredential(socket.retrieveProfile(address, unidentifiedAccess),

libsignal/service/src/main/java/org/whispersystems/signalservice/api/profiles/SignalServiceProfile.java

@@ -131,8 +131,6 @@ public boolean isStorage() {
   }
 
   public ProfileKeyCredentialResponse getProfileKeyCredentialResponse() {
-    if (!FeatureFlags.VERSIONED_PROFILES) return null;
-
     if (credential == null) return null;
 
     try {

libsignal/service/src/main/java/org/whispersystems/signalservice/internal/push/PushServiceSocket.java

@@ -598,10 +598,6 @@ public SignalServiceProfile retrieveProfileByUsername(String username, Optional<
   public ProfileAndCredential retrieveProfile(UUID target, ProfileKey profileKey, Optional<UnidentifiedAccess> unidentifiedAccess)
       throws NonSuccessfulResponseCodeException, PushNetworkException, VerificationFailedException
   {
-    if (!FeatureFlags.VERSIONED_PROFILES) {
-      throw new AssertionError();
-    }
-
     ProfileKeyVersion                  profileKeyIdentifier = profileKey.getProfileKeyVersion(target);
     ProfileKeyCredentialRequestContext requestContext       = clientZkProfileOperations.createProfileKeyCredentialRequestContext(random, target, profileKey);
     ProfileKeyCredentialRequest        request              = requestContext.getRequest();
@@ -636,7 +632,7 @@ public void retrieveProfileAvatar(String path, File destination, long maxSizeByt
   }
 
   public void setProfileName(String name) throws NonSuccessfulResponseCodeException, PushNetworkException {
-    if (FeatureFlags.VERSIONED_PROFILES) {
+    if (FeatureFlags.DISALLOW_OLD_PROFILE_SETTING) {
       throw new AssertionError();
     }
 
@@ -646,7 +642,7 @@ public void setProfileName(String name) throws NonSuccessfulResponseCodeExceptio
   public Optional<String> setProfileAvatar(ProfileAvatarData profileAvatar)
       throws NonSuccessfulResponseCodeException, PushNetworkException
   {
-    if (FeatureFlags.VERSIONED_PROFILES) {
+    if (FeatureFlags.DISALLOW_OLD_PROFILE_SETTING) {
       throw new AssertionError();
     }
 
@@ -680,10 +676,6 @@ public Optional<String> setProfileAvatar(ProfileAvatarData profileAvatar)
   public Optional<String> writeProfile(SignalServiceProfileWrite signalServiceProfileWrite, ProfileAvatarData profileAvatar)
       throws NonSuccessfulResponseCodeException, PushNetworkException
   {
-    if (!FeatureFlags.VERSIONED_PROFILES) {
-      throw new AssertionError();
-    }
-
     String                        requestBody    = JsonUtil.toJson(signalServiceProfileWrite);
     ProfileAvatarUploadAttributes formAttributes;