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

v30 regression: recurring logging of Link of notification is not an absolute URL and does not work in mobile and desktop clients #316

Closed
joshtrichards opened this issue Sep 24, 2024 · 0 comments · Fixed by #317

Comments

@joshtrichards
Copy link
Member

{"reqId":"DH567oSE5X6okdvhnDiW","level":2,"time":"2024-09-24T15:53:19+00:00","remoteAddr":"192.168.88.67","user":"ncadmin","app":"no app in context","method":"GET","url":"/ocs/v2.php/apps/notifications/api/v2/notifications","message":"Icon of notification is not an absolute URL and does not work in mobile and desktop clients [app: survey_client, subject: updated]","userAgent":"Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/115.0","version":"30.0.0.14","data":[],"id":"66f2e1cdf046c"}

{"reqId":"DH567oSE5X6okdvhnDiW","level":2,"time":"2024-09-24T15:53:19+00:00","remoteAddr":"192.168.88.67","user":"ncadmin","app":"no app in context","method":"GET","url":"/ocs/v2.php/apps/notifications/api/v2/notifications","message":"Link of notification is not an absolute URL and does not work in mobile and desktop clients [app: survey_client, subject: updated]","userAgent":"Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/115.0","version":"30.0.0.14","data":[],"id":"66f2e1cdf047b"}


{
  "reqId": "DH567oSE5X6okdvhnDiW",
  "level": 2,
  "time": "2024-09-24T15:53:19+00:00",
  "remoteAddr": "192.168.88.67",
  "user": "ncadmin",
  "app": "no app in context",
  "method": "GET",
  "url": "/ocs/v2.php/apps/notifications/api/v2/notifications",
  "message": "Link of notification is not an absolute URL and does not work in mobile and desktop clients [app: survey_client, subject: updated]",
  "userAgent": "Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/115.0",
  "version": "30.0.0.14",
  "data": [],
  "id": "66f2e1cdf047b"
}
joshtrichards added a commit that referenced this issue Sep 24, 2024
Fixes #316 

Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards joshtrichards changed the title v30 regression: Link of notification is not an absolute URL and does not work in mobile and desktop clients v30 regression: constant logging of Link of notification is not an absolute URL and does not work in mobile and desktop clients Sep 24, 2024
@joshtrichards joshtrichards changed the title v30 regression: constant logging of Link of notification is not an absolute URL and does not work in mobile and desktop clients v30 regression: recurring logging of Link of notification is not an absolute URL and does not work in mobile and desktop clients Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant