[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[tor-commits] [metrics-lib/release] Add descriptor digest to vote and streamline method names.
commit d242a2aa4d37ff0dcaf3b0093548ccb139c8ce46
Author: Karsten Loesing <karsten.loesing@xxxxxxx>
Date: Thu May 4 10:05:23 2017 +0200
Add descriptor digest to vote and streamline method names.
Implements #20333.
---
CHANGELOG.md | 2 +
.../org/torproject/descriptor/DirSourceEntry.java | 10 +++++
.../descriptor/DirectoryKeyCertificate.java | 10 +++++
.../torproject/descriptor/DirectorySignature.java | 11 +++++
.../torproject/descriptor/ExtraInfoDescriptor.java | 47 ++++++++++++++++++++++
.../org/torproject/descriptor/Microdescriptor.java | 11 +++++
.../torproject/descriptor/NetworkStatusEntry.java | 11 +++++
.../org/torproject/descriptor/RelayDirectory.java | 11 +++++
.../torproject/descriptor/RelayNetworkStatus.java | 11 +++++
.../descriptor/RelayNetworkStatusConsensus.java | 13 +++++-
.../descriptor/RelayNetworkStatusVote.java | 10 ++++-
.../torproject/descriptor/ServerDescriptor.java | 46 +++++++++++++++++++++
.../descriptor/impl/DirSourceEntryImpl.java | 5 +++
.../impl/DirectoryKeyCertificateImpl.java | 5 +++
.../descriptor/impl/DirectorySignatureImpl.java | 5 +++
.../descriptor/impl/ExtraInfoDescriptorImpl.java | 20 +++++++++
.../descriptor/impl/MicrodescriptorImpl.java | 5 +++
.../descriptor/impl/NetworkStatusEntryImpl.java | 5 +++
.../descriptor/impl/RelayDirectoryImpl.java | 5 +++
.../impl/RelayNetworkStatusConsensusImpl.java | 5 +++
.../descriptor/impl/RelayNetworkStatusImpl.java | 5 +++
.../impl/RelayNetworkStatusVoteImpl.java | 43 +++++++++++++++++++-
.../descriptor/impl/ServerDescriptorImpl.java | 20 +++++++++
.../impl/ExtraInfoDescriptorImplTest.java | 8 ++--
.../impl/RelayNetworkStatusConsensusImplTest.java | 2 +-
.../impl/RelayNetworkStatusVoteImplTest.java | 9 +++--
.../descriptor/impl/ServerDescriptorImplTest.java | 8 ++--
27 files changed, 328 insertions(+), 15 deletions(-)
diff --git a/CHANGELOG.md b/CHANGELOG.md
index e713a42..e636780 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -6,10 +6,12 @@
- Add support for six new key-value pairs added by OnionPerf.
- Fix a bug where DescriptorIndexCollector would not delete
extraneous local files if remote paths start with /.
+ - Add previously missing method to obtain the digest of a vote.
* Minor changes
- Accept extra arguments in statistics-related extra-info
descriptor lines, as permitted by dir-spec.txt.
+ - Streamline digest method names.
# Changes in version 1.6.0 - 2017-02-17
diff --git a/src/main/java/org/torproject/descriptor/DirSourceEntry.java b/src/main/java/org/torproject/descriptor/DirSourceEntry.java
index 49d7f8a..9996ff2 100644
--- a/src/main/java/org/torproject/descriptor/DirSourceEntry.java
+++ b/src/main/java/org/torproject/descriptor/DirSourceEntry.java
@@ -89,8 +89,18 @@ public interface DirSourceEntry {
* Return the SHA-1 vote digest, encoded as 40 lower-case hexadecimal
* characters.
*
+ * @deprecated Renamed to {@link #getVoteDigestSha1Hex()}.
+ *
* @since 1.0.0
*/
public String getVoteDigest();
+
+ /**
+ * Return the SHA-1 vote digest, encoded as 40 lower-case hexadecimal
+ * characters.
+ *
+ * @since 1.7.0
+ */
+ public String getVoteDigestSha1Hex();
}
diff --git a/src/main/java/org/torproject/descriptor/DirectoryKeyCertificate.java b/src/main/java/org/torproject/descriptor/DirectoryKeyCertificate.java
index 5a7877b..f2d37ad 100644
--- a/src/main/java/org/torproject/descriptor/DirectoryKeyCertificate.java
+++ b/src/main/java/org/torproject/descriptor/DirectoryKeyCertificate.java
@@ -102,8 +102,18 @@ public interface DirectoryKeyCertificate extends Descriptor {
* Return the SHA-1 certificate digest, encoded as 40 lower-case
* hexadecimal characters.
*
+ * @deprecated Renamed to {@link #getDigestSha1Hex()}.
+ *
* @since 1.0.0
*/
public String getCertificateDigest();
+
+ /**
+ * Return the SHA-1 certificate digest, encoded as 40 lower-case
+ * hexadecimal characters.
+ *
+ * @since 1.7.0
+ */
+ public String getDigestSha1Hex();
}
diff --git a/src/main/java/org/torproject/descriptor/DirectorySignature.java b/src/main/java/org/torproject/descriptor/DirectorySignature.java
index 0b1c993..bb3d5c4 100644
--- a/src/main/java/org/torproject/descriptor/DirectorySignature.java
+++ b/src/main/java/org/torproject/descriptor/DirectorySignature.java
@@ -37,11 +37,22 @@ public interface DirectorySignature {
* in the version 3 directory protocol, encoded as 40 upper-case
* hexadecimal characters.
*
+ * @deprecated Renamed to {@link #getSigningKeyDigestSha1Hex()}.
+ *
* @since 1.0.0
*/
public String getSigningKeyDigest();
/**
+ * Return the SHA-1 digest of the authority's medium-term signing key
+ * in the version 3 directory protocol, encoded as 40 upper-case
+ * hexadecimal characters.
+ *
+ * @since 1.7.0
+ */
+ public String getSigningKeyDigestSha1Hex();
+
+ /**
* Return the directory signature string made with the authority's
* identity key in the version 3 directory protocol.
*
diff --git a/src/main/java/org/torproject/descriptor/ExtraInfoDescriptor.java b/src/main/java/org/torproject/descriptor/ExtraInfoDescriptor.java
index cd63394..bb1cbe6 100644
--- a/src/main/java/org/torproject/descriptor/ExtraInfoDescriptor.java
+++ b/src/main/java/org/torproject/descriptor/ExtraInfoDescriptor.java
@@ -48,20 +48,43 @@ public interface ExtraInfoDescriptor extends Descriptor {
* characters, that is used to reference this descriptor from a server
* descriptor.
*
+ * @deprecated Renamed to {@link #getDigestSha1Hex()}.
+ *
* @since 1.0.0
*/
public String getExtraInfoDigest();
/**
+ * Return the SHA-1 descriptor digest, encoded as 40 lower-case (relay
+ * descriptors) or upper-case (bridge descriptors) hexadecimal
+ * characters, that is used to reference this descriptor from a server
+ * descriptor.
+ *
+ * @since 1.7.0
+ */
+ public String getDigestSha1Hex();
+
+ /**
* Return the SHA-256 descriptor digest, encoded as 43 base64
* characters without padding characters, that may be used to reference
* this descriptor from a server descriptor.
*
+ * @deprecated Renamed to {@link #getDigestSha256Base64()}.
+ *
* @since 1.1.0
*/
public String getExtraInfoDigestSha256();
/**
+ * Return the SHA-256 descriptor digest, encoded as 43 base64
+ * characters without padding characters, that may be used to reference
+ * this descriptor from a server descriptor.
+ *
+ * @since 1.7.0
+ */
+ public String getDigestSha256Base64();
+
+ /**
* Return the server's nickname consisting of 1 to 19 alphanumeric
* characters.
*
@@ -112,21 +135,45 @@ public interface ExtraInfoDescriptor extends Descriptor {
* upper-case hexadecimal characters, or null if no GeoIP database
* digest is included.
*
+ * @deprecated Renamed to {@link #getGeoipDbDigestSha1Hex()}.
+ *
* @since 1.0.0
*/
public String getGeoipDbDigest();
/**
+ * Return a SHA-1 digest of the GeoIP database file used by this server
+ * to resolve client IP addresses to country codes, encoded as 40
+ * upper-case hexadecimal characters, or null if no GeoIP database
+ * digest is included.
+ *
+ * @since 1.7.0
+ */
+ public String getGeoipDbDigestSha1Hex();
+
+ /**
* Return a SHA-1 digest of the GeoIPv6 database file used by this
* server to resolve client IP addresses to country codes, encoded as 40
* upper-case hexadecimal characters, or null if no GeoIPv6 database
* digest is included.
*
+ * @deprecated Renamed to {@link #getGeoip6DbDigestSha1Hex()}.
+ *
* @since 1.0.0
*/
public String getGeoip6DbDigest();
/**
+ * Return a SHA-1 digest of the GeoIPv6 database file used by this
+ * server to resolve client IP addresses to country codes, encoded as 40
+ * upper-case hexadecimal characters, or null if no GeoIPv6 database
+ * digest is included.
+ *
+ * @since 1.7.0
+ */
+ public String getGeoip6DbDigestSha1Hex();
+
+ /**
* Return the time in milliseconds since the epoch when the included
* directory request statistics interval ended, or -1 if no such
* statistics are included.
diff --git a/src/main/java/org/torproject/descriptor/Microdescriptor.java b/src/main/java/org/torproject/descriptor/Microdescriptor.java
index 155c799..b49b8bb 100644
--- a/src/main/java/org/torproject/descriptor/Microdescriptor.java
+++ b/src/main/java/org/torproject/descriptor/Microdescriptor.java
@@ -28,11 +28,22 @@ public interface Microdescriptor extends Descriptor {
* characters without padding characters, that is used to reference this
* descriptor from a vote or microdescriptor consensus.
*
+ * @deprecated Renamed to {@link #getDigestSha256Base64()}.
+ *
* @since 1.0.0
*/
public String getMicrodescriptorDigest();
/**
+ * Return the SHA-256 descriptor digest, encoded as 43 base64
+ * characters without padding characters, that is used to reference this
+ * descriptor from a vote or microdescriptor consensus.
+ *
+ * @since 1.7.0
+ */
+ public String getDigestSha256Base64();
+
+ /**
* Return the RSA-1024 public key in PEM format used to encrypt CREATE
* cells for this server, or null if the descriptor doesn't contain an
* onion key.
diff --git a/src/main/java/org/torproject/descriptor/NetworkStatusEntry.java b/src/main/java/org/torproject/descriptor/NetworkStatusEntry.java
index ba413bb..c59eab9 100644
--- a/src/main/java/org/torproject/descriptor/NetworkStatusEntry.java
+++ b/src/main/java/org/torproject/descriptor/NetworkStatusEntry.java
@@ -93,11 +93,22 @@ public interface NetworkStatusEntry {
* containing network status is a vote or microdesc consensus, or null
* otherwise.
*
+ * @deprecated Renamed to {@link #getMicrodescriptorDigestsSha256Base64()}.
+ *
* @since 1.0.0
*/
public Set<String> getMicrodescriptorDigests();
/**
+ * Return the (possibly empty) set of microdescriptor digests if the
+ * containing network status is a vote or microdesc consensus, or null
+ * otherwise.
+ *
+ * @since 1.7.0
+ */
+ public Set<String> getMicrodescriptorDigestsSha256Base64();
+
+ /**
* Return additional IP addresses and TCP ports where this server
* accepts TLS connections for the main OR protocol, or an empty list if
* the network status doesn't contain any such additional addresses and
diff --git a/src/main/java/org/torproject/descriptor/RelayDirectory.java b/src/main/java/org/torproject/descriptor/RelayDirectory.java
index fb07a41..dd696cf 100644
--- a/src/main/java/org/torproject/descriptor/RelayDirectory.java
+++ b/src/main/java/org/torproject/descriptor/RelayDirectory.java
@@ -97,8 +97,19 @@ public interface RelayDirectory extends Descriptor {
* hexadecimal characters, that the directory authority used to sign the
* directory.
*
+ * @deprecated Renamed to {@link #getDigestSha1Hex()}.
+ *
* @since 1.0.0
*/
public String getDirectoryDigest();
+
+ /**
+ * Return the SHA-1 directory digest, encoded as 40 lower-case
+ * hexadecimal characters, that the directory authority used to sign the
+ * directory.
+ *
+ * @since 1.7.0
+ */
+ public String getDigestSha1Hex();
}
diff --git a/src/main/java/org/torproject/descriptor/RelayNetworkStatus.java b/src/main/java/org/torproject/descriptor/RelayNetworkStatus.java
index 86361cd..b14d0c3 100644
--- a/src/main/java/org/torproject/descriptor/RelayNetworkStatus.java
+++ b/src/main/java/org/torproject/descriptor/RelayNetworkStatus.java
@@ -169,8 +169,19 @@ public interface RelayNetworkStatus extends Descriptor {
* characters, that the directory authority used to sign the network
* status.
*
+ * @deprecated Renamed to {@link #getDigestSha1Hex()}.
+ *
* @since 1.0.0
*/
public String getStatusDigest();
+
+ /**
+ * Return the SHA-1 status digest, encoded as 40 lower-case hexadecimal
+ * characters, that the directory authority used to sign the network
+ * status.
+ *
+ * @since 1.7.0
+ */
+ public String getDigestSha1Hex();
}
diff --git a/src/main/java/org/torproject/descriptor/RelayNetworkStatusConsensus.java b/src/main/java/org/torproject/descriptor/RelayNetworkStatusConsensus.java
index 57a8dd6..fe66007 100644
--- a/src/main/java/org/torproject/descriptor/RelayNetworkStatusConsensus.java
+++ b/src/main/java/org/torproject/descriptor/RelayNetworkStatusConsensus.java
@@ -282,12 +282,23 @@ public interface RelayNetworkStatusConsensus extends Descriptor {
public SortedMap<String, Integer> getBandwidthWeights();
/**
- * Return the SHA-1 digest of this consensus, encoded as 40 upper-case
+ * Return the SHA-1 digest of this consensus, encoded as 40 lower-case
* hexadecimal characters that directory authorities use to sign the
* consensus.
*
+ * @deprecated Renamed to {@link #getDigestSha1Hex()}.
+ *
* @since 1.0.0
*/
public String getConsensusDigest();
+
+ /**
+ * Return the SHA-1 digest of this consensus, encoded as 40 lower-case
+ * hexadecimal characters that directory authorities use to sign the
+ * consensus.
+ *
+ * @since 1.7.0
+ */
+ public String getDigestSha1Hex();
}
diff --git a/src/main/java/org/torproject/descriptor/RelayNetworkStatusVote.java b/src/main/java/org/torproject/descriptor/RelayNetworkStatusVote.java
index cfb012b..5d9d039 100644
--- a/src/main/java/org/torproject/descriptor/RelayNetworkStatusVote.java
+++ b/src/main/java/org/torproject/descriptor/RelayNetworkStatusVote.java
@@ -415,7 +415,7 @@ public interface RelayNetworkStatusVote extends Descriptor {
* @deprecated Removed in order to be more explicit that authorities may
* use different digest algorithms than "sha1"; see
* {@link #getSignatures()} and
- * {@link DirectorySignature#getSigningKeyDigest()} for
+ * {@link DirectorySignature#getSigningKeyDigestSha1Hex()} for
* alternatives.
*
* @since 1.0.0
@@ -491,5 +491,13 @@ public interface RelayNetworkStatusVote extends Descriptor {
* @since 1.3.0
*/
public List<DirectorySignature> getSignatures();
+
+ /**
+ * Return the SHA-1 digest of this vote, encoded as 40 lower-case hexadecimal
+ * characters that is used to reference this vote from a consensus.
+ *
+ * @since 1.7.0
+ */
+ public String getDigestSha1Hex();
}
diff --git a/src/main/java/org/torproject/descriptor/ServerDescriptor.java b/src/main/java/org/torproject/descriptor/ServerDescriptor.java
index 4ecade6..3ac0cc9 100644
--- a/src/main/java/org/torproject/descriptor/ServerDescriptor.java
+++ b/src/main/java/org/torproject/descriptor/ServerDescriptor.java
@@ -51,20 +51,43 @@ public interface ServerDescriptor extends Descriptor {
* characters, that is used to reference this descriptor from a network
* status descriptor.
*
+ * @deprecated Renamed to {@link #getDigestSha1Hex()}.
+ *
* @since 1.0.0
*/
public String getServerDescriptorDigest();
/**
+ * Return the SHA-1 descriptor digest, encoded as 40 lower-case (relay
+ * descriptors) or upper-case (bridge descriptors) hexadecimal
+ * characters, that is used to reference this descriptor from a network
+ * status descriptor.
+ *
+ * @since 1.7.0
+ */
+ public String getDigestSha1Hex();
+
+ /**
* Return the SHA-256 descriptor digest, encoded as 43 base64
* characters without padding characters, that may be used to reference
* this server descriptor from a network status descriptor.
*
+ * @deprecated Renamed to {@link #getDigestSha256Base64()}.
+ *
* @since 1.1.0
*/
public String getServerDescriptorDigestSha256();
/**
+ * Return the SHA-256 descriptor digest, encoded as 43 base64
+ * characters without padding characters, that may be used to reference
+ * this server descriptor from a network status descriptor.
+ *
+ * @since 1.7.0
+ */
+ public String getDigestSha256Base64();
+
+ /**
* Return the server's nickname consisting of 1 to 19 alphanumeric
* characters.
*
@@ -299,21 +322,44 @@ public interface ServerDescriptor extends Descriptor {
* encoded as 40 upper-case hexadecimal characters, or null if the
* server did not upload a corresponding extra-info descriptor.
*
+ * @deprecated Renamed to {@link #getExtraInfoDigestSha1Hex()}.
+ *
* @since 1.0.0
*/
public String getExtraInfoDigest();
/**
+ * Return the SHA-1 digest of the server's extra-info descriptor,
+ * encoded as 40 upper-case hexadecimal characters, or null if the
+ * server did not upload a corresponding extra-info descriptor.
+ *
+ * @since 1.7.0
+ */
+ public String getExtraInfoDigestSha1Hex();
+
+ /**
* Return the SHA-256 digest of the server's extra-info descriptor,
* encoded as 43 base64 characters without padding characters, or null
* if the server either did not upload a corresponding extra-info
* descriptor or did not refer to it using a SHA-256 digest.
*
+ * @deprecated Renamed to {@link #getExtraInfoDigestSha256Base64()}.
+ *
* @since 1.1.0
*/
public String getExtraInfoDigestSha256();
/**
+ * Return the SHA-256 digest of the server's extra-info descriptor,
+ * encoded as 43 base64 characters without padding characters, or null
+ * if the server either did not upload a corresponding extra-info
+ * descriptor or did not refer to it using a SHA-256 digest.
+ *
+ * @since 1.7.0
+ */
+ public String getExtraInfoDigestSha256Base64();
+
+ /**
* Return the list of hidden service descriptor version numbers that
* this server stores and serves, or null if it doesn't store and serve
* any hidden service descriptors.
diff --git a/src/main/java/org/torproject/descriptor/impl/DirSourceEntryImpl.java b/src/main/java/org/torproject/descriptor/impl/DirSourceEntryImpl.java
index 9e5995b..539d211 100644
--- a/src/main/java/org/torproject/descriptor/impl/DirSourceEntryImpl.java
+++ b/src/main/java/org/torproject/descriptor/impl/DirSourceEntryImpl.java
@@ -228,6 +228,11 @@ public class DirSourceEntryImpl implements DirSourceEntry {
@Override
public String getVoteDigest() {
+ return this.getVoteDigestSha1Hex();
+ }
+
+ @Override
+ public String getVoteDigestSha1Hex() {
return this.voteDigest;
}
}
diff --git a/src/main/java/org/torproject/descriptor/impl/DirectoryKeyCertificateImpl.java b/src/main/java/org/torproject/descriptor/impl/DirectoryKeyCertificateImpl.java
index df479f4..64df7aa 100644
--- a/src/main/java/org/torproject/descriptor/impl/DirectoryKeyCertificateImpl.java
+++ b/src/main/java/org/torproject/descriptor/impl/DirectoryKeyCertificateImpl.java
@@ -312,6 +312,11 @@ public class DirectoryKeyCertificateImpl extends DescriptorImpl
@Override
public String getCertificateDigest() {
+ return this.getDigestSha1Hex();
+ }
+
+ @Override
+ public String getDigestSha1Hex() {
return this.certificateDigest;
}
}
diff --git a/src/main/java/org/torproject/descriptor/impl/DirectorySignatureImpl.java b/src/main/java/org/torproject/descriptor/impl/DirectorySignatureImpl.java
index 995f525..7cf427a 100644
--- a/src/main/java/org/torproject/descriptor/impl/DirectorySignatureImpl.java
+++ b/src/main/java/org/torproject/descriptor/impl/DirectorySignatureImpl.java
@@ -109,6 +109,11 @@ public class DirectorySignatureImpl implements DirectorySignature {
@Override
public String getSigningKeyDigest() {
+ return this.getSigningKeyDigestSha1Hex();
+ }
+
+ @Override
+ public String getSigningKeyDigestSha1Hex() {
return this.signingKeyDigest;
}
diff --git a/src/main/java/org/torproject/descriptor/impl/ExtraInfoDescriptorImpl.java b/src/main/java/org/torproject/descriptor/impl/ExtraInfoDescriptorImpl.java
index bbaae19..e2b3827 100644
--- a/src/main/java/org/torproject/descriptor/impl/ExtraInfoDescriptorImpl.java
+++ b/src/main/java/org/torproject/descriptor/impl/ExtraInfoDescriptorImpl.java
@@ -889,6 +889,11 @@ public abstract class ExtraInfoDescriptorImpl extends DescriptorImpl
@Override
public String getExtraInfoDigest() {
+ return this.getDigestSha1Hex();
+ }
+
+ @Override
+ public String getDigestSha1Hex() {
return this.extraInfoDigest;
}
@@ -896,6 +901,11 @@ public abstract class ExtraInfoDescriptorImpl extends DescriptorImpl
@Override
public String getExtraInfoDigestSha256() {
+ return this.getDigestSha256Base64();
+ }
+
+ @Override
+ public String getDigestSha256Base64() {
return this.extraInfoDigestSha256;
}
@@ -938,6 +948,11 @@ public abstract class ExtraInfoDescriptorImpl extends DescriptorImpl
@Override
public String getGeoipDbDigest() {
+ return this.getGeoipDbDigestSha1Hex();
+ }
+
+ @Override
+ public String getGeoipDbDigestSha1Hex() {
return this.geoipDbDigest;
}
@@ -945,6 +960,11 @@ public abstract class ExtraInfoDescriptorImpl extends DescriptorImpl
@Override
public String getGeoip6DbDigest() {
+ return this.getGeoip6DbDigestSha1Hex();
+ }
+
+ @Override
+ public String getGeoip6DbDigestSha1Hex() {
return this.geoip6DbDigest;
}
diff --git a/src/main/java/org/torproject/descriptor/impl/MicrodescriptorImpl.java b/src/main/java/org/torproject/descriptor/impl/MicrodescriptorImpl.java
index b199a20..5c94371 100644
--- a/src/main/java/org/torproject/descriptor/impl/MicrodescriptorImpl.java
+++ b/src/main/java/org/torproject/descriptor/impl/MicrodescriptorImpl.java
@@ -264,6 +264,11 @@ public class MicrodescriptorImpl extends DescriptorImpl
@Override
public String getMicrodescriptorDigest() {
+ return this.getDigestSha256Base64();
+ }
+
+ @Override
+ public String getDigestSha256Base64() {
return this.microdescriptorDigest;
}
diff --git a/src/main/java/org/torproject/descriptor/impl/NetworkStatusEntryImpl.java b/src/main/java/org/torproject/descriptor/impl/NetworkStatusEntryImpl.java
index f67e6bd..152546a 100644
--- a/src/main/java/org/torproject/descriptor/impl/NetworkStatusEntryImpl.java
+++ b/src/main/java/org/torproject/descriptor/impl/NetworkStatusEntryImpl.java
@@ -337,6 +337,11 @@ public class NetworkStatusEntryImpl implements NetworkStatusEntry {
@Override
public Set<String> getMicrodescriptorDigests() {
+ return this.getMicrodescriptorDigestsSha256Base64();
+ }
+
+ @Override
+ public Set<String> getMicrodescriptorDigestsSha256Base64() {
return this.microdescriptorDigests == null ? null
: new HashSet<>(this.microdescriptorDigests);
}
diff --git a/src/main/java/org/torproject/descriptor/impl/RelayDirectoryImpl.java b/src/main/java/org/torproject/descriptor/impl/RelayDirectoryImpl.java
index ff32934..fc8f07b 100644
--- a/src/main/java/org/torproject/descriptor/impl/RelayDirectoryImpl.java
+++ b/src/main/java/org/torproject/descriptor/impl/RelayDirectoryImpl.java
@@ -556,6 +556,11 @@ public class RelayDirectoryImpl extends DescriptorImpl
@Override
public String getDirectoryDigest() {
+ return this.getDigestSha1Hex();
+ }
+
+ @Override
+ public String getDigestSha1Hex() {
return this.directoryDigest;
}
}
diff --git a/src/main/java/org/torproject/descriptor/impl/RelayNetworkStatusConsensusImpl.java b/src/main/java/org/torproject/descriptor/impl/RelayNetworkStatusConsensusImpl.java
index dcb8938..ef135d4 100644
--- a/src/main/java/org/torproject/descriptor/impl/RelayNetworkStatusConsensusImpl.java
+++ b/src/main/java/org/torproject/descriptor/impl/RelayNetworkStatusConsensusImpl.java
@@ -397,6 +397,11 @@ public class RelayNetworkStatusConsensusImpl extends NetworkStatusImpl
@Override
public String getConsensusDigest() {
+ return this.getDigestSha1Hex();
+ }
+
+ @Override
+ public String getDigestSha1Hex() {
return this.consensusDigest;
}
diff --git a/src/main/java/org/torproject/descriptor/impl/RelayNetworkStatusImpl.java b/src/main/java/org/torproject/descriptor/impl/RelayNetworkStatusImpl.java
index 7c01e3c..143540c 100644
--- a/src/main/java/org/torproject/descriptor/impl/RelayNetworkStatusImpl.java
+++ b/src/main/java/org/torproject/descriptor/impl/RelayNetworkStatusImpl.java
@@ -299,6 +299,11 @@ public class RelayNetworkStatusImpl extends NetworkStatusImpl
@Override
public String getStatusDigest() {
+ return this.getDigestSha1Hex();
+ }
+
+ @Override
+ public String getDigestSha1Hex() {
return this.statusDigest;
}
diff --git a/src/main/java/org/torproject/descriptor/impl/RelayNetworkStatusVoteImpl.java b/src/main/java/org/torproject/descriptor/impl/RelayNetworkStatusVoteImpl.java
index 1f64173..05f5cc2 100644
--- a/src/main/java/org/torproject/descriptor/impl/RelayNetworkStatusVoteImpl.java
+++ b/src/main/java/org/torproject/descriptor/impl/RelayNetworkStatusVoteImpl.java
@@ -7,6 +7,9 @@ import org.torproject.descriptor.DescriptorParseException;
import org.torproject.descriptor.DirectorySignature;
import org.torproject.descriptor.RelayNetworkStatusVote;
+import java.io.UnsupportedEncodingException;
+import java.security.MessageDigest;
+import java.security.NoSuchAlgorithmException;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.HashSet;
@@ -18,6 +21,7 @@ import java.util.SortedMap;
import java.util.SortedSet;
import java.util.TreeMap;
import java.util.TreeSet;
+import javax.xml.bind.DatatypeConverter;
/* Contains a network status vote. */
public class RelayNetworkStatusVoteImpl extends NetworkStatusImpl
@@ -64,6 +68,36 @@ public class RelayNetworkStatusVoteImpl extends NetworkStatusImpl
this.checkAtLeastOnceKeywords(atLeastOnceKeywords);
this.checkFirstKeyword("network-status-version");
this.clearParsedKeywords();
+ this.calculateDigest();
+ }
+
+ private void calculateDigest() throws DescriptorParseException {
+ try {
+ String ascii = new String(this.getRawDescriptorBytes(), "US-ASCII");
+ String startToken = "network-status-version ";
+ String sigToken = "\ndirectory-signature ";
+ if (!ascii.contains(sigToken)) {
+ return;
+ }
+ int start = ascii.indexOf(startToken);
+ int sig = ascii.indexOf(sigToken) + sigToken.length();
+ if (start >= 0 && sig >= 0 && sig > start) {
+ byte[] forDigest = new byte[sig - start];
+ System.arraycopy(this.getRawDescriptorBytes(), start,
+ forDigest, 0, sig - start);
+ this.digestSha1Hex = DatatypeConverter.printHexBinary(
+ MessageDigest.getInstance("SHA-1").digest(forDigest))
+ .toLowerCase();
+ }
+ } catch (UnsupportedEncodingException e) {
+ /* Handle below. */
+ } catch (NoSuchAlgorithmException e) {
+ /* Handle below. */
+ }
+ if (this.digestSha1Hex == null) {
+ throw new DescriptorParseException("Could not calculate vote "
+ + "digest.");
+ }
}
protected void parseHeader(byte[] headerBytes)
@@ -615,6 +649,13 @@ public class RelayNetworkStatusVoteImpl extends NetworkStatusImpl
}
}
+ private String digestSha1Hex;
+
+ @Override
+ public String getDigestSha1Hex() {
+ return this.digestSha1Hex;
+ }
+
private String nickname;
@Override
@@ -769,7 +810,7 @@ public class RelayNetworkStatusVoteImpl extends NetworkStatusImpl
for (DirectorySignature signature : this.signatures) {
if (DirectorySignatureImpl.DEFAULT_ALGORITHM.equals(
signature.getAlgorithm())) {
- signingKeyDigest = signature.getSigningKeyDigest();
+ signingKeyDigest = signature.getSigningKeyDigestSha1Hex();
break;
}
}
diff --git a/src/main/java/org/torproject/descriptor/impl/ServerDescriptorImpl.java b/src/main/java/org/torproject/descriptor/impl/ServerDescriptorImpl.java
index 309cad4..ac113d2 100644
--- a/src/main/java/org/torproject/descriptor/impl/ServerDescriptorImpl.java
+++ b/src/main/java/org/torproject/descriptor/impl/ServerDescriptorImpl.java
@@ -743,6 +743,11 @@ public abstract class ServerDescriptorImpl extends DescriptorImpl
@Override
public String getServerDescriptorDigest() {
+ return this.getDigestSha1Hex();
+ }
+
+ @Override
+ public String getDigestSha1Hex() {
return this.serverDescriptorDigest;
}
@@ -750,6 +755,11 @@ public abstract class ServerDescriptorImpl extends DescriptorImpl
@Override
public String getServerDescriptorDigestSha256() {
+ return this.getDigestSha256Base64();
+ }
+
+ @Override
+ public String getDigestSha256Base64() {
return this.serverDescriptorDigestSha256;
}
@@ -933,6 +943,11 @@ public abstract class ServerDescriptorImpl extends DescriptorImpl
@Override
public String getExtraInfoDigest() {
+ return this.getExtraInfoDigestSha1Hex();
+ }
+
+ @Override
+ public String getExtraInfoDigestSha1Hex() {
return this.extraInfoDigest;
}
@@ -940,6 +955,11 @@ public abstract class ServerDescriptorImpl extends DescriptorImpl
@Override
public String getExtraInfoDigestSha256() {
+ return this.getExtraInfoDigestSha256Base64();
+ }
+
+ @Override
+ public String getExtraInfoDigestSha256Base64() {
return this.extraInfoDigestSha256;
}
diff --git a/src/test/java/org/torproject/descriptor/impl/ExtraInfoDescriptorImplTest.java b/src/test/java/org/torproject/descriptor/impl/ExtraInfoDescriptorImplTest.java
index 118ada8..76a2148 100644
--- a/src/test/java/org/torproject/descriptor/impl/ExtraInfoDescriptorImplTest.java
+++ b/src/test/java/org/torproject/descriptor/impl/ExtraInfoDescriptorImplTest.java
@@ -1115,7 +1115,7 @@ public class ExtraInfoDescriptorImplTest {
.createWithGeoipDbDigestLine("geoip-db-digest "
+ "916A3CA8B7DF61473D5AE5B21711F35F301CE9E8");
assertEquals("916A3CA8B7DF61473D5AE5B21711F35F301CE9E8",
- descriptor.getGeoipDbDigest());
+ descriptor.getGeoipDbDigestSha1Hex());
}
@Test(expected = DescriptorParseException.class)
@@ -1155,7 +1155,7 @@ public class ExtraInfoDescriptorImplTest {
.createWithGeoip6DbDigestLine("geoip6-db-digest "
+ "916A3CA8B7DF61473D5AE5B21711F35F301CE9E8");
assertEquals("916A3CA8B7DF61473D5AE5B21711F35F301CE9E8",
- descriptor.getGeoip6DbDigest());
+ descriptor.getGeoip6DbDigestSha1Hex());
}
@Test()
@@ -1969,7 +1969,7 @@ public class ExtraInfoDescriptorImplTest {
RelayExtraInfoDescriptor descriptor =
new RelayExtraInfoDescriptorImpl(descriptorBytes, true);
assertEquals("Pt1BtzfRwhYqGCDo8jjchS8nJP3ovrDyHGn+dqPbMgw",
- descriptor.getExtraInfoDigestSha256());
+ descriptor.getDigestSha256Base64());
}
@Test()
@@ -1993,7 +1993,7 @@ public class ExtraInfoDescriptorImplTest {
BridgeExtraInfoDescriptor descriptor =
new BridgeExtraInfoDescriptorImpl(descriptorBytes, true);
assertEquals("TvrqpjI7OmCtwGwair/NHUxg5ROVVQYz6/EDyXsDHR4",
- descriptor.getExtraInfoDigestSha256());
+ descriptor.getDigestSha256Base64());
}
}
diff --git a/src/test/java/org/torproject/descriptor/impl/RelayNetworkStatusConsensusImplTest.java b/src/test/java/org/torproject/descriptor/impl/RelayNetworkStatusConsensusImplTest.java
index b150a6a..6298668 100644
--- a/src/test/java/org/torproject/descriptor/impl/RelayNetworkStatusConsensusImplTest.java
+++ b/src/test/java/org/torproject/descriptor/impl/RelayNetworkStatusConsensusImplTest.java
@@ -389,7 +389,7 @@ public class RelayNetworkStatusConsensusImplTest {
if ("14C131DFC5C6F93646BE72FA1401C02A8DF2E8B4".equals(
signature.getIdentity())) {
assertEquals("3509BA5A624403A905C74DA5C8A0CEC9E0D3AF86",
- signature.getSigningKeyDigest());
+ signature.getSigningKeyDigestSha1Hex());
}
}
assertEquals(285, (int) consensus.getBandwidthWeights().get("Wbd"));
diff --git a/src/test/java/org/torproject/descriptor/impl/RelayNetworkStatusVoteImplTest.java b/src/test/java/org/torproject/descriptor/impl/RelayNetworkStatusVoteImplTest.java
index 727d707..4a61f1c 100644
--- a/src/test/java/org/torproject/descriptor/impl/RelayNetworkStatusVoteImplTest.java
+++ b/src/test/java/org/torproject/descriptor/impl/RelayNetworkStatusVoteImplTest.java
@@ -734,7 +734,7 @@ public class RelayNetworkStatusVoteImplTest {
assertEquals("80550987E1D626E3EBA5E5E75A458DE0626D088C",
signature.getIdentity());
assertEquals("EEB9299D295C1C815E289FBF2F2BBEA5F52FDD19",
- signature.getSigningKeyDigest());
+ signature.getSigningKeyDigestSha1Hex());
assertEquals("-----BEGIN SIGNATURE-----\n"
+ "iHEU3Iidya5RIrjyYgv8tlU0R+rF56/3/MmaaZi0a67e7ZkISfQ4dghScHxn"
+ "F3Yh\nrXVaaoP07r6Ta+s0g1Zijm3lms50Nk/4tV2p8Y63c3F4Q3DAnK40Oi"
@@ -1512,15 +1512,18 @@ public class RelayNetworkStatusVoteImplTest {
assertEquals("sha256", firstSignature.getAlgorithm());
assertEquals(identitySha256, firstSignature.getIdentity());
assertEquals(signingKeyDigestSha256,
- firstSignature.getSigningKeyDigest());
+ firstSignature.getSigningKeyDigestSha1Hex());
assertEquals(signatureSha256 + "\n", firstSignature.getSignature());
DirectorySignature secondSignature = vote.getSignatures().get(1);
assertEquals("sha1", secondSignature.getAlgorithm());
assertEquals(identitySha1, secondSignature.getIdentity());
assertEquals(signingKeyDigestSha1,
- secondSignature.getSigningKeyDigest());
+ secondSignature.getSigningKeyDigestSha1Hex());
assertEquals(signatureSha1 + "\n", secondSignature.getSignature());
assertEquals(signingKeyDigestSha1, vote.getSigningKeyDigest());
+ System.out.println(new String(vote.getRawDescriptorBytes()));
+ assertEquals("c0d58c8d3c3695526f6eb5c0d9f8452b2234d303",
+ vote.getDigestSha1Hex());
}
@Test()
diff --git a/src/test/java/org/torproject/descriptor/impl/ServerDescriptorImplTest.java b/src/test/java/org/torproject/descriptor/impl/ServerDescriptorImplTest.java
index b713dd6..4a84764 100644
--- a/src/test/java/org/torproject/descriptor/impl/ServerDescriptorImplTest.java
+++ b/src/test/java/org/torproject/descriptor/impl/ServerDescriptorImplTest.java
@@ -472,7 +472,7 @@ public class ServerDescriptorImplTest {
assertEquals(51200, (int) descriptor.getBandwidthBurst());
assertEquals(53470, (int) descriptor.getBandwidthObserved());
assertEquals("1469D1550738A25B1E7B47CDDBCD7B2899F51B74",
- descriptor.getExtraInfoDigest());
+ descriptor.getExtraInfoDigestSha1Hex());
assertEquals(Arrays.asList(new Integer[] {2}),
descriptor.getHiddenServiceDirVersions());
assertEquals("Random Person <nobody AT example dot com>",
@@ -828,7 +828,7 @@ public class ServerDescriptorImplTest {
.createWithExtraInfoDigestLine("extra-info-digest "
+ "1469D1550738A25B1E7B47CDDBCD7B2899F51B74");
assertEquals("1469D1550738A25B1E7B47CDDBCD7B2899F51B74",
- descriptor.getExtraInfoDigest());
+ descriptor.getExtraInfoDigestSha1Hex());
}
@Test(expected = DescriptorParseException.class)
@@ -858,7 +858,7 @@ public class ServerDescriptorImplTest {
throws DescriptorParseException {
ServerDescriptor descriptor = DescriptorBuilder
.createWithExtraInfoDigestLine(null);
- assertNull(descriptor.getExtraInfoDigest());
+ assertNull(descriptor.getExtraInfoDigestSha1Hex());
}
@Test()
@@ -871,7 +871,7 @@ public class ServerDescriptorImplTest {
extraInfoDigest, additionalExtraInfoDigest);
ServerDescriptor descriptor = DescriptorBuilder
.createWithExtraInfoDigestLine(extraInfoDigestLine);
- assertEquals(extraInfoDigest, descriptor.getExtraInfoDigest());
+ assertEquals(extraInfoDigest, descriptor.getExtraInfoDigestSha1Hex());
}
@Test()
_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits