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

Ocis vs ldap. No notifications to user #6184

Closed
Tracked by #6230
ScharfViktor opened this issue Apr 28, 2023 · 1 comment · Fixed by #6211
Closed
Tracked by #6230

Ocis vs ldap. No notifications to user #6184

ScharfViktor opened this issue Apr 28, 2023 · 1 comment · Fixed by #6211
Assignees
Labels
Priority:p1-urgent Consider a hotfix release with only that fix Type:Bug

Comments

@ScharfViktor
Copy link
Contributor

#6105 (comment)

ocis against ldap using https://github.com/owncloud/ocis/blob/master/deployments/examples/ocis_ldap/docker-compose.yml

  • admin shares file to einstein
  • einstein logs in

Expect: einstein has notification

Actual: No notification. 502 error
Screenshot 2023-04-27 at 14 22 17

log:

{"level":"error","service":"proxy","error":"not found","service":"com.owncloud.userlog.userlog","time":"2023-04-27T12:17:29.423137876Z","line":"github.com/owncloud/ocis/v2/services/proxy/pkg/router/router.go:139","message":"could not select service from the registry"}


2023/04/27 12:17:29 http: proxy error: unsupported protocol scheme ""

{"level":"info","service":"proxy","proto":"HTTP/1.1","request-id":"9651f55d-9ade-4abc-b5cc-2359cd6a079e","remote-addr":"172.29.0.1","method":"GET","status":502,"path":"/ocs/v2.php/apps/notifications/api/v1/notifications","duration":10.633583,"bytes":0,"time":"2023-04-27T12:17:29.423246334Z","line":"github.com/owncloud/ocis/v2/services/proxy/pkg/middleware/accesslog.go:28","message":"access-log"}

cc @kobergj

@ScharfViktor ScharfViktor mentioned this issue Apr 28, 2023
89 tasks
@micbar micbar added the Priority:p1-urgent Consider a hotfix release with only that fix label May 2, 2023
@micbar micbar moved this from Qualification to Prio 1 in Infinite Scale Team Board May 2, 2023
@2403905 2403905 self-assigned this May 2, 2023
@rhafer
Copy link
Contributor

rhafer commented May 2, 2023

Just stumbled over this. I think this is because the ocis_ldap deployment uses OCIS_RUN_SERVICES (https://github.com/owncloud/ocis/blob/master/deployments/examples/ocis_ldap/docker-compose.yml#LL81C7-L81C24) to avoid starting the "idm" service. However, the recently introduced userlog service is not included in that list.

I think we should remove OCIS_RUN_SERVICES from the example and use OCIS_EXCLUDE_RUN_SERVICES=idm instead. Makes things more obivous.

2403905 added a commit to 2403905/ocis that referenced this issue May 3, 2023
2403905 added a commit that referenced this issue May 3, 2023
the idm run excluded for the ocis ldap stack #6184
@github-project-automation github-project-automation bot moved this from Prio 1 to Done in Infinite Scale Team Board May 3, 2023
ownclouders pushed a commit that referenced this issue May 3, 2023
the idm run excluded for the ocis ldap stack #6184
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p1-urgent Consider a hotfix release with only that fix Type:Bug
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants