Skip to content

Commit

Permalink
Release v5.0.0 (#135)
Browse files Browse the repository at this point in the history
* Update Alpha info

* Expose AttestedClient

* Cherry-pick Expose AttestedClient

* Update Changelog and MobileCoin version
  • Loading branch information
dolanbernard authored May 26, 2023
1 parent 5059ac5 commit f654797
Show file tree
Hide file tree
Showing 22 changed files with 49 additions and 46 deletions.
19 changes: 10 additions & 9 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,16 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [5.0.0] - 2023-05-25

### Changed
- Exposed `AttestedClient` and related APIs
- Updated dependencies
- Updated MobileCoin to version 5.0.0

### Upgrading
No code changes are *required* to upgrade from 4.1.1 to 5.0.0

## [4.1.1] - 2023-04-26

### Added
Expand All @@ -16,7 +26,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Moved Util.getSharedSecret to OnetimeKeys and made public

### Upgrading

No code changes are *required* to upgrade from 4.1.0 to 4.1.1

## [4.1.0] - 2023-04-06
Expand All @@ -36,7 +45,6 @@ No code changes are *required* to upgrade from 4.1.0 to 4.1.1
- Dependency updates

### Upgrading

No code changes are *required* to upgrade from 4.0.0.1 to 4.1.0

## [4.0.0.1] - 2023-02-10
Expand Down Expand Up @@ -75,7 +83,6 @@ No code changes are *required* to upgrade from 4.0.0.1 to 4.1.0
- Fixed connection reset errors in HTTP clients

### Upgrading

No code changes are *required* to upgrade from 1.2.2.4 to 4.0.0

## [1.2.2.4] - 2022-08-24
Expand All @@ -89,7 +96,6 @@ No code changes are *required* to upgrade from 1.2.2.4 to 4.0.0
- Query size calculation in `DefaultFogQueryScalingStrategy`

### Upgrading

No code changes are *required* to upgrade from 1.2.2.3 to 1.2.2.4

## [1.2.2.3] - 2022-08-22
Expand All @@ -99,7 +105,6 @@ No code changes are *required* to upgrade from 1.2.2.3 to 1.2.2.4
- Added `getTransactionStatusQuick` to `MobileCoinClient`

### Upgrading

No code changes are *required* to upgrade from 1.2.2.2 to 1.2.2.3

## [1.2.2.2] - 2022-08-11
Expand All @@ -108,7 +113,6 @@ No code changes are *required* to upgrade from 1.2.2.2 to 1.2.2.3
- `GRPCFogKeyImageService` properly converts `StatusRuntimeException` to `NetworkException`

### Upgrading

No code changes are *required* to upgrade from 1.2.2.1 to 1.2.2.2

## [1.2.2.1] - 2022-07-21
Expand All @@ -119,7 +123,6 @@ No code changes are *required* to upgrade from 1.2.2.1 to 1.2.2.2
- Changed visibility of `OwnedTxOut.getAmount()` to public

### Upgrading

No code changes are *required* to upgrade from 1.2.2 to 1.2.2.1

## [1.2.2] - 2022-07-21
Expand All @@ -138,7 +141,6 @@ some issues caused by `OwnedTxOut`s being updated after being fetched from the p
- Fixed a dependency issue introduced by some project structure changes

### Upgrading

No code changes are *required* to upgrade from 1.2.1 to 1.2.2

- To easily handle various types of transaction failure differently, code such as the following
Expand All @@ -154,7 +156,6 @@ value of `MobileCoinTransactionClient.submitTransaction`
- Updated bindings to version 1.2.1

### Upgrading

No code changes are *required* to upgrade from 1.2.0 to 1.2.1

- Calls to `new Amount(value, TokenId.MOB)` may be replaced with `Amount.ofMOB(value)`.
Expand Down
2 changes: 1 addition & 1 deletion android-sdk/publish.gradle
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
apply plugin: 'maven-publish'

version '4.1.1'
version '5.0.0'
group 'com.mobilecoin'

Properties properties = new Properties()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@
import static org.junit.Assert.assertNotSame;
import static org.junit.Assert.assertNull;

import com.mobilecoin.lib.log.Logger;

import org.junit.Test;

import java.math.BigInteger;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,7 @@ private static Set<X509Certificate> getTestTrustRoots() {

@NonNull
private static Set<X509Certificate> getDevTrustRoots() {
String trustRootBase64String = "MIIG1TCCBL2gAwIBAgIQbFWr29AHksedBwzYEZ7WvzANBgkqhkiG9w0BAQwFADCBiDELMAkGA1UEBhMCVVMxEzARBgNVBAgTCk5ldyBKZXJzZXkxFDASBgNVBAcTC0plcnNleSBDaXR5MR4wHAYDVQQKExVUaGUgVVNFUlRSVVNUIE5ldHdvcmsxLjAsBgNVBAMTJVVTRVJUcnVzdCBSU0EgQ2VydGlmaWNhdGlvbiBBdXRob3JpdHkwHhcNMjAwMTMwMDAwMDAwWhcNMzAwMTI5MjM1OTU5WjBLMQswCQYDVQQGEwJBVDEQMA4GA1UEChMHWmVyb1NTTDEqMCgGA1UEAxMhWmVyb1NTTCBSU0EgRG9tYWluIFNlY3VyZSBTaXRlIENBMIICIjANBgkqhkiG9w0BAQEFAAOCAg8AMIICCgKCAgEAhmlzfqO1Mdgj4W3dpBPTVBX1AuvcAyG1fl0dUnw/MeueCWzRWTheZ35LVo91kLI3DDVaZKW+TBAsJBjEbYmMwcWSTWYCg5334SF0+ctDAsFxsX+rTDh9kSrG/4mp6OShubLaEIUJiZo4t873TuSd0Wj5DWt3DtpAG8T35l/v+xrN8ub8PSSoX5Vkgw+jWf4KQtNvUFLDq8mFWhUnPL6jHAADXpvs4lTNYwOtx9yQtbpxwSt7QJY1+ICrmRJB6BuKRt/jfDJF9JscRQVlHIxQdKAJl7oaVnXgDkqtk2qddd3kCDXd74gv813G91z7CjsGyJ93oJIlNS3UgFbD6V54JMgZ3rSmotYbz98oZxX7MKbtCm1aJ/q+hTv2YK1yMxrnfcieKmOYBbFDhnW5O6RMA703dBK92j6XRN2EttLkQuujZgy+jXRKtaWMIlkNkWJmOiHmErQngHvtiNkIcjJumq1ddFX4iaTI40a6zgvIBtxFeDs2RfcaH73er7ctNUUqgQT5rFgJhMmFx76rQgB5OZUkodb5k2ex7P+Gu4J86bS15094UuYcV09hVeknmTh5Ex9CBKipLS2W2wKBakf+aVYnNCU6S0nASqt2xrZpGC1v7v6DhuepyyJtn3qSV2PoBiU5Sql+aARpwUibQMGm44gjyNDqDlVp+ShLQlUH9x8CAwEAAaOCAXUwggFxMB8GA1UdIwQYMBaAFFN5v1qqK0rPVIDh2JvAnfKyA2bLMB0GA1UdDgQWBBTI2XhootkZaNU9ct5fCj7ctYaGpjAOBgNVHQ8BAf8EBAMCAYYwEgYDVR0TAQH/BAgwBgEB/wIBADAdBgNVHSUEFjAUBggrBgEFBQcDAQYIKwYBBQUHAwIwIgYDVR0gBBswGTANBgsrBgEEAbIxAQICTjAIBgZngQwBAgEwUAYDVR0fBEkwRzBFoEOgQYY/aHR0cDovL2NybC51c2VydHJ1c3QuY29tL1VTRVJUcnVzdFJTQUNlcnRpZmljYXRpb25BdXRob3JpdHkuY3JsMHYGCCsGAQUFBwEBBGowaDA/BggrBgEFBQcwAoYzaHR0cDovL2NydC51c2VydHJ1c3QuY29tL1VTRVJUcnVzdFJTQUFkZFRydXN0Q0EuY3J0MCUGCCsGAQUFBzABhhlodHRwOi8vb2NzcC51c2VydHJ1c3QuY29tMA0GCSqGSIb3DQEBDAUAA4ICAQAVDwoIzQDVercT0eYqZjBNJ8VNWwVFlQOtZERqn5iWnEVaLZZdzxlbvz2Fx0ExUNuUEgYkIVM4YocKkCQ7hO5noicoq/DrEYH5IuNcuW1I8JJZ9DLuB1fYvIHlZ2JG46iNbVKA3ygAEz86RvDQlt2C494qqPVItRjrz9YlJEGT0DrttyApq0YLFDzf+Z1pkMhh7c+7fXeJqmIhfJpduKc8HEQkYQQShen426S3H0JrIAbKcBCiyYFuOhfyvuwVCFDfFvrjADjd4jX1uQXd161IyFRbm89s2Oj5oU1wDYz5sx+hoCuh6lSs+/uPuWomIq3y1GDFNafW+LsHBU16lQo5Q2yh25laQsKRgyPmMpHJ98edm6y2sHUabASmRHxvGiuwwE25aDU02SAeepyImJ2CzB80YG7WxlynHqNhpE7xfC7PzQlLgmfEHdU+tHFeQazRQnrFkW2WkqRGIq7cKRnyypvjPMkjeiV9lRdAM9fSJvsB3svUuu1coIG1xxI1yegoGM4r5QP4RGIVvYaiI76C0djoSbQ/dkIUUXQuB8AL5jyH34g3BZaaXyvpmnV4ilppMXVAnAYGON51WhJ6W0xNdNJwzYASZYH+tmCWI+N60Gv2NNMGHwMZ7e9bXgzUCZH5FaBFDGR5S9VWqHB73Q+OyIVvIbKYcSc2w/aSuFKGSA==";
String trustRootBase64String = "MIIFjDCCA3SgAwIBAgINAgO8UKMnU/CRgCLt8TANBgkqhkiG9w0BAQsFADBHMQswCQYDVQQGEwJVUzEiMCAGA1UEChMZR29vZ2xlIFRydXN0IFNlcnZpY2VzIExMQzEUMBIGA1UEAxMLR1RTIFJvb3QgUjEwHhcNMjAwODEzMDAwMDQyWhcNMjcwOTMwMDAwMDQyWjBGMQswCQYDVQQGEwJVUzEiMCAGA1UEChMZR29vZ2xlIFRydXN0IFNlcnZpY2VzIExMQzETMBEGA1UEAxMKR1RTIENBIDFQNTCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBALOC8CSMvy2Hr7LZp676yrpE1ls+/rL3smUW3N4Q6E8tEFhaKIaHoe5qs6DZdU9/oVIBi1WoSlsGSMg2EiWrifnyI1+dYGX5XNq+OuhcbX2c0IQYhTDNTpvsPNiz4ZbU88ULZduPsHTL9h7zePGslcXdc8MxiIGvdKpv/QzjBZXwxRBPZWP6oK/GGD3Fod+XedcFibMwsHSuPZIQa4wVd90LBFf7gQPd6iI01eVWsvDEjUGxwwLbYuyA0P921IbkBBq2tgwrYnF92a/Z8V76wB7KoBlcVfCA0SoMB4aQnzXjKCtb7yPIox2kozru/oPcgkwlsE3FUa2em9NbhMIaWukCAwEAAaOCAXYwggFyMA4GA1UdDwEB/wQEAwIBhjAdBgNVHSUEFjAUBggrBgEFBQcDAQYIKwYBBQUHAwIwEgYDVR0TAQH/BAgwBgEB/wIBADAdBgNVHQ4EFgQU1fyeDd8eyt0Il5duK8VfxSv17LgwHwYDVR0jBBgwFoAU5K8rJnEaK0gnhS9SZizv8IkTcT4waAYIKwYBBQUHAQEEXDBaMCYGCCsGAQUFBzABhhpodHRwOi8vb2NzcC5wa2kuZ29vZy9ndHNyMTAwBggrBgEFBQcwAoYkaHR0cDovL3BraS5nb29nL3JlcG8vY2VydHMvZ3RzcjEuZGVyMDQGA1UdHwQtMCswKaAnoCWGI2h0dHA6Ly9jcmwucGtpLmdvb2cvZ3RzcjEvZ3RzcjEuY3JsME0GA1UdIARGMEQwOAYKKwYBBAHWeQIFAzAqMCgGCCsGAQUFBwIBFhxodHRwczovL3BraS5nb29nL3JlcG9zaXRvcnkvMAgGBmeBDAECATANBgkqhkiG9w0BAQsFAAOCAgEAbGMn7iPf5VJoTYFmkYXffWXlWzcxCCayB12avrHKAbmtv5139lEd15jFC0mhe6HX02jlRA+LujbdQoJ30o3d9T/768gHmJPuWtC1Pd5LHC2MTex+jHv+TkD98LSzWQIQUVzjwCv9twZIUX4JXj8P3Kf+l+d5xQ5EiXjFaVkpoJo6SDYpppSTVS24R7XplrWfB82mqz4yisCGg8XBQcifLzWODcAHeuGsyWW1y4qn3XHYYWU5hKwyPvd6NvFWn1epQW1akKfbOup1gAxjC2l0bwdMFfM3KKUZpG719iDNY7J+xCsJdYna0Twuck82GqGeRNDNm6YjCD+XoaeeWqX3CZStXXZdKFbRGmZRUQd73j2wyO8weiQtvrizhvZL9/C1T//Oxvn2PyonCA8JPiNax+NCLXo25D2YlmA5mOrR22Mq63gJsU4hs463zj6S8ZVcpDnQwCvIUxX10i+CzQZ0Z5mQdzcKly3FHB700FvpFePqAgnIE9cTcGW/+4ibWiW+dwnhp2pOEXW5Hk3xABtqZnmOw27YbaIiom0F+yzy8VDloNHYnzV9/HCrWSoC8b6w0/H4zRK5aiWQW+OFIOb12stAHBk0IANhd7p/SA9JCynr52Fkx2PRR+sc4e6URu85c8zuTyuN3PtYp7NlIJmVuftVb9eWbpQ99HqSjmMd320=";
byte[] trustRootBytes = Base64.decode(trustRootBase64String, Base64.DEFAULT);
return Util.makeCertificatesFromData(trustRootBytes);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ public GRPCAttestedService(@NonNull ManagedChannel managedChannel,

@NonNull
@Override
AttestedApiGrpc.AttestedApiBlockingStub newBlockingStub(@NonNull ManagedChannel managedChannel) {
protected AttestedApiGrpc.AttestedApiBlockingStub newBlockingStub(@NonNull ManagedChannel managedChannel) {
return AttestedApiGrpc.newBlockingStub(managedChannel);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public GRPCBlockchainService(@NonNull ManagedChannel managedChannel,

@NonNull
@Override
BlockchainAPIGrpc.BlockchainAPIBlockingStub newBlockingStub(@NonNull ManagedChannel managedChannel) {
protected BlockchainAPIGrpc.BlockchainAPIBlockingStub newBlockingStub(@NonNull ManagedChannel managedChannel) {
return BlockchainAPIGrpc.newBlockingStub(managedChannel);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public GRPCConsensusClientService(@NonNull ManagedChannel managedChannel,

@NonNull
@Override
ConsensusClientAPIGrpc.ConsensusClientAPIBlockingStub newBlockingStub(@NonNull ManagedChannel managedChannel) {
protected ConsensusClientAPIGrpc.ConsensusClientAPIBlockingStub newBlockingStub(@NonNull ManagedChannel managedChannel) {
return ConsensusClientAPIGrpc.newBlockingStub(managedChannel);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ public GRPCFogBlockService(@NonNull ManagedChannel managedChannel,

@NonNull
@Override
FogBlockAPIGrpc.FogBlockAPIBlockingStub newBlockingStub(@NonNull ManagedChannel managedChannel) {
protected FogBlockAPIGrpc.FogBlockAPIBlockingStub newBlockingStub(@NonNull ManagedChannel managedChannel) {
return FogBlockAPIGrpc.newBlockingStub(managedChannel);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ public GRPCFogKeyImageService(@NonNull ManagedChannel managedChannel,

@NonNull
@Override
FogKeyImageAPIGrpc.FogKeyImageAPIBlockingStub newBlockingStub(
protected FogKeyImageAPIGrpc.FogKeyImageAPIBlockingStub newBlockingStub(
@NonNull ManagedChannel managedChannel
) {
return FogKeyImageAPIGrpc.newBlockingStub(managedChannel);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ public GRPCFogMerkleProofService(@NonNull ManagedChannel managedChannel,

@NonNull
@Override
FogMerkleProofAPIGrpc.FogMerkleProofAPIBlockingStub newBlockingStub(
protected FogMerkleProofAPIGrpc.FogMerkleProofAPIBlockingStub newBlockingStub(
@NonNull ManagedChannel managedChannel
) {
return FogMerkleProofAPIGrpc.newBlockingStub(managedChannel);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ public GRPCFogReportService(@NonNull ManagedChannel managedChannel,

@NonNull
@Override
ReportAPIGrpc.ReportAPIBlockingStub newBlockingStub(@NonNull ManagedChannel managedChannel) {
protected ReportAPIGrpc.ReportAPIBlockingStub newBlockingStub(@NonNull ManagedChannel managedChannel) {
return ReportAPIGrpc.newBlockingStub(managedChannel);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public GRPCFogUntrustedService(@NonNull ManagedChannel managedChannel,

@NonNull
@Override
FogUntrustedTxOutApiGrpc.FogUntrustedTxOutApiBlockingStub newBlockingStub(
protected FogUntrustedTxOutApiGrpc.FogUntrustedTxOutApiBlockingStub newBlockingStub(
@NonNull ManagedChannel managedChannel
) {
return FogUntrustedTxOutApiGrpc.newBlockingStub(getManagedChannel());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ public GRPCFogViewService(@NonNull ManagedChannel managedChannel,

@NonNull
@Override
FogViewAPIGrpc.FogViewAPIBlockingStub newBlockingStub(@NonNull ManagedChannel managedChannel) {
protected FogViewAPIGrpc.FogViewAPIBlockingStub newBlockingStub(@NonNull ManagedChannel managedChannel) {
return FogViewAPIGrpc.newBlockingStub(getManagedChannel());
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,13 +11,13 @@
import io.grpc.ManagedChannel;
import io.grpc.stub.AbstractStub;

abstract class GRPCService <T extends AbstractStub<T>> implements ApiService {
public abstract class GRPCService <T extends AbstractStub<T>> implements ApiService {
private static final int MEGABYTE = 1024 * 1024;
private static final int MAX_INBOUND_MESSAGE_SIZE = 50 * MEGABYTE;
private final T apiBlockingStub;
private final ManagedChannel managedChannel;

GRPCService(@NonNull ManagedChannel managedChannel,
protected GRPCService(@NonNull ManagedChannel managedChannel,
@NonNull CookieInterceptor cookieInterceptor,
@NonNull AuthInterceptor authInterceptor,
@NonNull ExecutorService executorService) {
Expand All @@ -31,10 +31,10 @@ abstract class GRPCService <T extends AbstractStub<T>> implements ApiService {
}

@NonNull
abstract T newBlockingStub(@NonNull ManagedChannel managedChannel);
protected abstract T newBlockingStub(@NonNull ManagedChannel managedChannel);

@NonNull
T getApiBlockingStub() {
protected T getApiBlockingStub() {
return apiBlockingStub;
}

Expand Down
4 changes: 2 additions & 2 deletions android-sdk/src/main/java/com/mobilecoin/lib/AnyClient.java
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ protected AnyClient(@NonNull LoadBalancer loadBalancer,
}

@NonNull
final ServiceAPIManager getAPIManager() {
protected final ServiceAPIManager getAPIManager() {
return this.serviceAPIManager;
}

Expand Down Expand Up @@ -74,7 +74,7 @@ private MobileCoinUri getNextServiceUri() {
}

@NonNull
final ClientConfig.Service getServiceConfig() {
public final ClientConfig.Service getServiceConfig() {
return serviceConfig;
}

Expand Down
22 changes: 11 additions & 11 deletions android-sdk/src/main/java/com/mobilecoin/lib/AttestedClient.java
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
* Base class for attested communication with View/Ledger/Consensus servers
*/

abstract class AttestedClient extends AnyClient {
public abstract class AttestedClient extends AnyClient {
private final static String TAG = AttestedClient.class.getName();
// How long to wait for the managed connection to gracefully shutdown in milliseconds

Expand All @@ -41,13 +41,13 @@ protected AttestedClient(@NonNull LoadBalancer loadBalancer,
/**
* Attest service connection or throw an exception if error occurs.
*/
protected abstract void attest(@NonNull Transport transport)
public abstract void attest(@NonNull Transport transport)
throws AttestationException, NetworkException;

/**
* Reset service connection
*/
protected synchronized void deattest() {
public synchronized void deattest() {
Logger.i(TAG, "De-attesting the managed channel");
attestReset();
}
Expand All @@ -58,7 +58,7 @@ protected synchronized void deattest() {
*
* @return whether or no the client is attested
*/
protected synchronized boolean isAttested() {
public synchronized boolean isAttested() {
Logger.d(TAG, "Is channel attested?", null, (rustObj != 0) ? "Yes" : "No");
return (rustObj != 0);
}
Expand All @@ -69,7 +69,7 @@ protected synchronized boolean isAttested() {
* @return encrypted {@link Attest.Message}
*/
@NonNull
protected synchronized Attest.Message encryptMessage(
public synchronized Attest.Message encryptMessage(
@Nullable AbstractMessageLite<?, ?> message,
@Nullable AbstractMessageLite<?, ?> aadMessage
) throws AttestationException {
Expand All @@ -93,7 +93,7 @@ protected synchronized Attest.Message encryptMessage(

@Override
@NonNull
synchronized Transport getNetworkTransport() throws NetworkException, AttestationException {
public synchronized Transport getNetworkTransport() throws NetworkException, AttestationException {
Transport transport = super.getNetworkTransport();
if(!isAttested()) {
attest(transport);
Expand All @@ -102,7 +102,7 @@ synchronized Transport getNetworkTransport() throws NetworkException, Attestatio
}

@NonNull
protected synchronized Attest.Message encryptMessage(@NonNull AbstractMessageLite<?, ?> message)
public synchronized Attest.Message encryptMessage(@NonNull AbstractMessageLite<?, ?> message)
throws AttestationException {
return encryptMessage(message, null);
}
Expand All @@ -113,7 +113,7 @@ protected synchronized Attest.Message encryptMessage(@NonNull AbstractMessageLit
* @return decrypted {@link Attest.Message}
*/
@NonNull
protected synchronized Attest.Message decryptMessage(@NonNull Attest.Message message)
public synchronized Attest.Message decryptMessage(@NonNull Attest.Message message)
throws AttestationException {
Logger.i(TAG, "Decrypt response message");
try {
Expand All @@ -138,7 +138,7 @@ protected synchronized Attest.Message decryptMessage(@NonNull Attest.Message mes
* @param serviceUri must include the port as well
*/
@NonNull
protected byte[] attestStart(@NonNull MobileCoinUri serviceUri) throws AttestationException {
public byte[] attestStart(@NonNull MobileCoinUri serviceUri) throws AttestationException {
Logger.i(TAG, "FFI: attest_start call");
try {
ResponderId responderId;
Expand All @@ -163,7 +163,7 @@ protected byte[] attestStart(@NonNull MobileCoinUri serviceUri) throws Attestati
*
* @param authResponse an auth response obtained from the attested service
*/
protected void attestFinish(@NonNull byte[] authResponse,
public void attestFinish(@NonNull byte[] authResponse,
@NonNull Verifier attestVerifier
) throws AttestationException {
Logger.i(TAG, "FFI: attest_finish call");
Expand All @@ -187,7 +187,7 @@ protected void attestFinish(@NonNull byte[] authResponse,
* It is needed to dispose of the invalid pending state, if the attestation was unsuccessful
* (i.e. attestation server is unreachable)
*/
protected synchronized void attestReset() {
public synchronized void attestReset() {
Logger.i(TAG, "Reset attested state");
resetNetworkTransport();
if (rustObj != 0) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ class AttestedConsensusClient extends AttestedClient {
* @param transport a channel that requires attestation
*/
@Override
protected synchronized void attest(@NonNull Transport transport)
public synchronized void attest(@NonNull Transport transport)
throws AttestationException, NetworkException {
try {
Logger.i(TAG, "Attest consensus connection");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ class AttestedLedgerClient extends AttestedClient {
* @param transport a channel that requires attestation
*/
@Override
protected synchronized void attest(@NonNull Transport transport)
public synchronized void attest(@NonNull Transport transport)
throws AttestationException, NetworkException {
try {
Logger.i(TAG, "Attest ledger connection");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ class AttestedViewClient extends AttestedClient {
* @param transport a channel that requires attestation
*/
@Override
protected synchronized void attest(@NonNull Transport transport)
public synchronized void attest(@NonNull Transport transport)
throws AttestationException, NetworkException {
try {
Logger.i(TAG, "Attest view connection");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
import com.mobilecoin.lib.network.uri.MobileCoinUri;

/** Balances server load for MobileCoin services. */
interface LoadBalancer {
public interface LoadBalancer {

/** Returns a new service {@link MobileCoinUri} upon each invocation. */
MobileCoinUri getNextServiceUri();
Expand Down
Loading

0 comments on commit f654797

Please sign in to comment.