Skip to content

do not use ocClient for anonymous connections#328

Merged
tobiasKaminsky merged 1 commit intomasterfrom
push
Aug 20, 2019
Merged

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

Conversation

@tobiasKaminsky
Copy link
Member

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

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@nextcloud-android-bot
Copy link
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
Copy link

codecov bot commented Aug 19, 2019

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
Copy link

codecov bot commented Aug 19, 2019

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
Copy link

codecov bot commented Aug 19, 2019

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

Comments