Skip to content

Commit 3ba6d00

Browse files
committed
Fix compatibility issues and update documentation for next hotfix release
1 parent 68110b9 commit 3ba6d00

File tree

5 files changed

+12
-3
lines changed

5 files changed

+12
-3
lines changed

docs/PowerAuth-Server-0.22.0.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -313,3 +313,7 @@ The [VerifyOfflineSignature](./SOAP-Service-Methods.md#method-verifyofflinesigna
313313
### External User Identifier
314314

315315
The [CommitActivation](./SOAP-Service-Methods.md#method-commitactivation), [RemoveActivation](./SOAP-Service-Methods.md#method-removeactivation), [BlockActivation](./SOAP-Service-Methods.md#method-blockactivation) and [UnblockActivation](./SOAP-Service-Methods.md#method-unblockactivation) methods have been updated to allow specification of external user identifier. The related PowerAuth client methods have been updated to reflect the change of parameters. If you use any of these methods, please update the SOAP method call.
316+
317+
### Revoking Recovery Codes on Activation Removal
318+
319+
We added an optional `revokeRecoveryCodes` attribute to [activation removal service call](./SOAP-Service-Methods.md#method-removeactivation). This flag indicates if recovery codes that are associated with removed activation should be also revoked. By default, the value of the flag is `false`, hence omitting the flag results in the same behavior as before this change.

docs/SOAP-Service-Methods.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -486,6 +486,7 @@ Remove activation with given ID. This operation is irreversible. Activations can
486486
|------|------|-------------|
487487
| `String` | `activationId` | An identifier of an activation |
488488
| `String` | `externalUserId` | User ID of user who removed the activation. Use null value if activation owner caused the change. |
489+
| `Boolean` | `revokeRecoveryCodes` | An optional flag that indicates if recovery codes, that were created in the scope of the removed activation, should be also revoked. |
489490

490491
#### Response
491492

powerauth-java-client-axis/src/main/java/io/getlime/security/powerauth/soap/axis/client/PowerAuthServiceClient.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -398,7 +398,7 @@ public PowerAuthPortV3ServiceStub.RemoveActivationResponse removeActivation(Stri
398398
* @return {@link io.getlime.powerauth.soap.v3.PowerAuthPortV3ServiceStub.RemoveActivationResponse}
399399
* @throws RemoteException In case of a business logic error.
400400
*/
401-
public PowerAuthPortV3ServiceStub.RemoveActivationResponse removeActivation(String activationId, String externalUserId, boolean revokeRecoveryCodes) throws RemoteException {
401+
public PowerAuthPortV3ServiceStub.RemoveActivationResponse removeActivation(String activationId, String externalUserId, Boolean revokeRecoveryCodes) throws RemoteException {
402402
PowerAuthPortV3ServiceStub.RemoveActivationRequest request = new PowerAuthPortV3ServiceStub.RemoveActivationRequest();
403403
request.setActivationId(activationId);
404404
request.setExternalUserId(externalUserId);

powerauth-java-client-spring/src/main/java/io/getlime/security/powerauth/soap/spring/client/PowerAuthServiceClient.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,7 @@ public RemoveActivationResponse removeActivation(String activationId, String ext
268268
* @param revokeRecoveryCodes Indicates if the recovery codes associated with this activation should be also revoked.
269269
* @return {@link RemoveActivationResponse}
270270
*/
271-
public RemoveActivationResponse removeActivation(String activationId, String externalUserId, boolean revokeRecoveryCodes) {
271+
public RemoveActivationResponse removeActivation(String activationId, String externalUserId, Boolean revokeRecoveryCodes) {
272272
RemoveActivationRequest request = new RemoveActivationRequest();
273273
request.setActivationId(activationId);
274274
request.setExternalUserId(externalUserId);

powerauth-java-server/src/main/java/io/getlime/security/powerauth/app/server/service/v3/PowerAuthServiceImpl.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -404,7 +404,11 @@ public RemoveActivationResponse removeActivation(RemoveActivationRequest request
404404
try {
405405
String activationId = request.getActivationId();
406406
String externalUserId = request.getExternalUserId();
407-
boolean revokeRecoveryCodes = request.isRevokeRecoveryCodes();
407+
Boolean revokeRecoveryCodes = request.isRevokeRecoveryCodes();
408+
if (revokeRecoveryCodes == null) {
409+
// The default value is false for revokeRecoveryCodes
410+
revokeRecoveryCodes = false;
411+
}
408412
logger.info("RemoveActivationRequest received, activation ID: {}, revoke recovery codes: {}", activationId, revokeRecoveryCodes);
409413
RemoveActivationResponse response = behavior.getActivationServiceBehavior().removeActivation(activationId, externalUserId, revokeRecoveryCodes);
410414
logger.info("RemoveActivationRequest succeeded");

0 commit comments

Comments
 (0)