Skip to content
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

[stable30] fix: make sure we have a valid scheme when testing ocm urls #49900

Merged
merged 1 commit into from
Dec 17, 2024

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Dec 17, 2024

From f08d053

{
  "reqId": "Z1HCXyNZzFr7XuNp4Tw13QAAAA4",
  "level": 2,
  "time": "2024-12-05T15:10:23+00:00",
  "remoteAddr": "1.2.3.4",
  "user": "admin",
  "app": "no app in context",
  "method": "PROPFIND",
  "url": "/remote.php/dav/files/admin/test-share/",
  "message": "error while discovering ocm provider",
  "userAgent": "",
  "version": "28.0.10.2",
  "exception": {
    "Exception": "OCP\\Http\\Client\\LocalServerException",
    "Message": "Could not detect any host",
    "Code": 0,
    "Trace": [
      {
        "file": "/home/www/nextcloud/lib/private/Http/Client/Client.php",
        "line": 229,
        "function": "preventLocalAddress",
        "class": "OC\\Http\\Client\\Client",
        "type": "->",
        "args": [
          "nextcloud.test.de/ocm-provider/",
          [
            10,
            true,
            10
          ]
        ]
      },

@skjnldsv skjnldsv self-assigned this Dec 17, 2024
@skjnldsv skjnldsv requested review from ArtificialOwl, a team, icewind1991 and sorbaugh and removed request for a team December 17, 2024 14:25
@skjnldsv
Copy link
Member Author

/backport to stable29

@skjnldsv skjnldsv enabled auto-merge December 17, 2024 14:31
@skjnldsv skjnldsv merged commit 8e7ce5c into stable30 Dec 17, 2024
170 checks passed
@skjnldsv skjnldsv deleted the fix/ocm-host branch December 17, 2024 15:46
@come-nc
Copy link
Contributor

come-nc commented Dec 17, 2024

Needs a backport to master I suppose?

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.

4 participants