-
Notifications
You must be signed in to change notification settings - Fork 6
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
P2P API #4
Conversation
} | ||
|
||
//@NonNull | ||
public TLSIdentity getIdentity( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing throws CouchbaseLiteException.
} | ||
|
||
//@NonNull | ||
public TLSIdentity createIdentity( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing throws CouchbaseLiteException.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be static method.
} | ||
|
||
//@NonNull | ||
public TLSIdentity importIdentity( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing throws CouchbaseLiteException.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be static method.
return null; | ||
} | ||
|
||
public void deleteIdentity(@NonNull String alias) { } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing throws CouchbaseLiteException.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be static method.
@@ -937,7 +937,7 @@ private boolean filterDocument( | |||
long dict, | |||
boolean isPush) { | |||
final ReplicationFilter filter = (isPush) ? config.getPushFilter() : config.getPullFilter(); | |||
return filter.filtered(new Document(getDatabase(), docId, revId, new FLDict(dict)), flags); | |||
return (filter != null) && filter.filtered(new Document(getDatabase(), docId, revId, new FLDict(dict)), flags); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch.
private Date expiration; | ||
|
||
@Nullable | ||
protected List<Certificate> getCerts() { return certs; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should it be public?
@Nullable | ||
protected Date getExpiration() { return expiration; } | ||
|
||
protected void setExpiration(@Nullable Date expiration) { this.expiration = expiration; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
expiration is readonly.
@Nullable | ||
protected List<Certificate> getCerts() { return certs; } | ||
|
||
protected void setCerts(@Nullable List<Certificate> certs) { this.certs = certs; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
certs is readonly.
public class TLSIdentity extends AbstractTLSIdentity { | ||
|
||
@Nullable | ||
public TLSIdentity getIdentity(KeyStore keyStore, String alias, byte[] keyPassword) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing throws CouchbaseLiteException.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
keyPassword is nullable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be static method.
return null; | ||
} | ||
|
||
public TLSIdentity createIdentity( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing throws CouchbaseLiteException.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be static method.
public class TLSIdentity extends AbstractTLSIdentity { | ||
|
||
@Nullable | ||
public TLSIdentity getIdentity(@NonNull String alias, @Nullable byte[] keyPassword) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be static method.
} | ||
|
||
//@NonNull | ||
public TLSIdentity createIdentity( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be static method.
return null; | ||
} | ||
|
||
public void deleteIdentity(@NonNull String alias) { } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be static method.
} | ||
|
||
//@NonNull | ||
public TLSIdentity importIdentity( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be static method.
All errors repaired. |
this is PR https://github.com/couchbaselabs/couchbase-lite-java-ee-root/pull/6
but working...
Implementation of the P2P API.