Blackfire "agent socket" not reachable as 127.0.0.1 (in Local Beach)

[kdambekalns]

The update to Blackfire v2 works fine if the BLACKFIRE_AGENT_SOCKET is set correctly in Local Beach. Otherwise you get a hard-to-debug message

Are you authorized to profile this page? No probe response, missing PHP extension or invalid signature for relaying agent.

when trying to debug HTTP requests (blackfire run inside Local Beach works fine!)

[kdambekalns]

Observations for Local Beach

These are needed environment variables for the blackfire/blackfire container:

      - BLACKFIRE_CLIENT_ID=${BLACKFIRE_CLIENT_ID}
      - BLACKFIRE_CLIENT_TOKEN=${BLACKFIRE_CLIENT_TOKEN}
      - BLACKFIRE_SERVER_ID=${BLACKFIRE_SERVER_ID}
      - BLACKFIRE_SERVER_TOKEN=${BLACKFIRE_SERVER_TOKEN}

That's what is needed, says their documentation. One can add these to get more logging and remove a warning:

      - BLACKFIRE_LOG_LEVEL=3
      - BLACKFIRE_DISABLE_LEGACY_PORT=true

These are needed environment variables for the flownative/beach-php container:

      - BEACH_ADDON_BLACKFIRE_ENABLE=true
      - BLACKFIRE_CLIENT_ID=${BLACKFIRE_CLIENT_ID}
      - BLACKFIRE_CLIENT_TOKEN=…${BLACKFIRE_CLIENT_TOKEN}
      - BLACKFIRE_SERVER_ID=${BLACKFIRE_SERVER_ID}
      - BLACKFIRE_SERVER_TOKEN=${BLACKFIRE_SERVER_TOKEN}
      - BLACKFIRE_AGENT_SOCKET=tcp://<project-name>_blackfire.local_beach:8307

---

Internally, it's a bit weird right now… BEACH_ADDON_BLACKFIRE_SERVER_* are set from BLACKFIRE_SERVER_* if not defined:

docker-beach-php/root-files/opt/flownative/lib/beach.sh

Lines 59 to 60 in d7941ac

	export BEACH_ADDON_BLACKFIRE_SERVER_ID=${BEACH_ADDON_BLACKFIRE_SERVER_ID:-${BLACKFIRE_SERVER_ID:-}}
	export BEACH_ADDON_BLACKFIRE_SERVER_TOKEN=${BEACH_ADDON_BLACKFIRE_SERVER_TOKEN:-${BLACKFIRE_SERVER_TOKEN:-}}

And then BLACKFIRE_SERVER_* are (re-)set from BEACH_ADDON_BLACKFIRE_SERVER_*:

docker-beach-php/root-files/opt/flownative/lib/beach.sh

Lines 65 to 66 in d7941ac

	export BLACKFIRE_SERVER_ID=${BEACH_ADDON_BLACKFIRE_SERVER_ID}
	export BLACKFIRE_SERVER_TOKEN=${BEACH_ADDON_BLACKFIRE_SERVER_TOKEN}

I suggest to drop BEACH_ADDON_BLACKFIRE_SERVER_* completely.