open-telemetry / open-telemetry/opentelemetry-php

[opentelemetry-php-contrib] Add Twig instrumentation

Open
#1,815 8 comments 0 reactions 0 assignees View on GitHub

A pull request for this has already been merged.

  • #8563 by @cmelius — merged
Dominant language
PHP
Stars
912
Forks
232
Avg merge
7d 16h
Merged PRs (30d)
4

Description

in my company we needed a deeper look at how it used twig templates and where it's wasting time.
It is a website running on CraftCMS / PHP 8.4.

I could not really find good traces with the standard instrumentations so I built my own
https://github.com/cmelius/opentelemetry-auto-twig

I am not really sure if this is needed here, if everything was done right. I coded it fast. There may be things missing, but it's working for me.
Just wanted to drop it here in case somebody could use it.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the linked pull request #8563 and the referenced opentelemetry-auto-twig repository, since the issue names no files or tests in this project. First establish whether Twig instrumentation belongs here and what gaps remain; done would require an agreed scope and project-specific implementation and validation.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
observability
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.