Skip to content

do not use ocClient for anonymous connections#328

Merged
tobiasKaminsky merged 1 commit into
masterfrom
push
Aug 20, 2019
Merged

do not use ocClient for anonymous connections#328
tobiasKaminsky merged 1 commit into
masterfrom
push

Conversation

@tobiasKaminsky

Copy link
Copy Markdown
Member

Signed-off-by: tobiasKaminsky tobias@kaminsky.me

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

Lint

TypemasterPR
Warnings00
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings14
Correctness Warnings40
Internationalization Warnings6
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings3
Performance Warnings16
Security Warnings17
Dodgy code Warnings81
Total181

SpotBugs (master)

Warning TypeNumber
Bad practice Warnings14
Correctness Warnings40
Internationalization Warnings6
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings3
Performance Warnings16
Security Warnings17
Dodgy code Warnings81
Total181

@codecov

codecov Bot commented Aug 19, 2019

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (master@0a13fa2). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #328   +/-   ##
=========================================
  Coverage          ?   21.58%           
=========================================
  Files             ?      117           
  Lines             ?     5166           
  Branches          ?      701           
=========================================
  Hits              ?     1115           
  Misses            ?     3921           
  Partials          ?      130
Impacted Files Coverage Δ
...egisterAccountDeviceForNotificationsOperation.java 0% <ø> (ø)
...ions/UnregisterAccountDeviceForProxyOperation.java 0% <0%> (ø)
...ations/RegisterAccountDeviceForProxyOperation.java 0% <0%> (ø)

2 similar comments
@codecov

codecov Bot commented Aug 19, 2019

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (master@0a13fa2). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #328   +/-   ##
=========================================
  Coverage          ?   21.58%           
=========================================
  Files             ?      117           
  Lines             ?     5166           
  Branches          ?      701           
=========================================
  Hits              ?     1115           
  Misses            ?     3921           
  Partials          ?      130
Impacted Files Coverage Δ
...egisterAccountDeviceForNotificationsOperation.java 0% <ø> (ø)
...ions/UnregisterAccountDeviceForProxyOperation.java 0% <0%> (ø)
...ations/RegisterAccountDeviceForProxyOperation.java 0% <0%> (ø)

@codecov

codecov Bot commented Aug 19, 2019

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (master@0a13fa2). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #328   +/-   ##
=========================================
  Coverage          ?   21.58%           
=========================================
  Files             ?      117           
  Lines             ?     5166           
  Branches          ?      701           
=========================================
  Hits              ?     1115           
  Misses            ?     3921           
  Partials          ?      130
Impacted Files Coverage Δ
...egisterAccountDeviceForNotificationsOperation.java 0% <ø> (ø)
...ions/UnregisterAccountDeviceForProxyOperation.java 0% <0%> (ø)
...ations/RegisterAccountDeviceForProxyOperation.java 0% <0%> (ø)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants