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

Create an 'admin service' in PHP #34

Closed
austinlparker opened this issue May 23, 2022 · 8 comments
Closed

Create an 'admin service' in PHP #34

austinlparker opened this issue May 23, 2022 · 8 comments
Assignees
Labels
enhancement New feature or request

Comments

@austinlparker
Copy link
Member

Objective

Create an 'admin service' written in PHP for the web store.

The following features should be supported:

  • Authentication for 'admin users'
  • UI for CRUD operations on items in the product catalog.
  • API for CRUD operations on items in the product catalog.
@cartersocha
Copy link
Contributor

Linking to the issue in the php repo: open-telemetry/opentelemetry-php#658

@bobstrecansky should I assign the current issue owner to this as well?

@bobstrecansky
Copy link

Sounds good!

@DebajitDas
Copy link
Member

As discussed in the #otel-community-demo slack channel, we can also use Apache WebServer to host 'Admin Service' in PHP through mod_php. Apache instrumentation is already available to generate spans and traces.

@tidal
Copy link
Member

tidal commented Jun 2, 2022

@DebajitDas
Thank you.
However we most likely would prefer not to use Apache, as that's a bit outdated, or in fact any web-sever (Nginx,, Caddy, etc.) as a runtime, but a modern faster runtime as Swoole or Roadrunner.

@cartersocha
Copy link
Contributor

Hey @tidal, @bobstrecansky, is there an update on this? We merged the rust service today, so PHP is the remaining outstanding language not present.

@cartersocha cartersocha added enhancement New feature or request and removed required-for-release labels Jul 14, 2022
@tidal
Copy link
Member

tidal commented Jul 25, 2022

As discussed with @cartersocha I will create a PR with basic service setup as a draft.
This service will feature roadrunner with OpenTelemetry middleware as an app server.
API Platform will make the implementation of auth and API easy.
It also allows to generate an admin ui from the API.

@julianocosta89
Copy link
Member

julianocosta89 commented Oct 6, 2022

@austinlparker can we have this ticket closed?
I believe we do not need an admin service anymore, as PHP was covered in the Quote Service.

@cartersocha
Copy link
Contributor

Closing this!

smith pushed a commit to smith/opentelemetry-demo that referenced this issue Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Development

No branches or pull requests

6 participants