Skip to content
Open
Show file tree
Hide file tree
Changes from 8 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
!/tracer/vendor
tmp/
composer.lock*
!tests/Frameworks/Symfony/Version_6_2/composer.lock
**/core*
.DS_Store
.scenarios.lock/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ class Symfony62Tests {
baseTag: 'apache2-mod-php',
phpVersion: phpVersion,
phpVariant: variant,
www: 'symfony62',
www: '../../../tests/Frameworks/Symfony/Version_6_2',
)

@Test
Expand Down Expand Up @@ -205,12 +205,20 @@ class Symfony62Tests {
endpoints.size() > 0
})

assert endpoints.size() == 6
assert endpoints.size() == 14
assert endpoints.find { it.path == '/' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /' } != null
assert endpoints.find { it.path == '/dynamic-path/{param01}' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /dynamic-path/{param01}' } != null
assert endpoints.find { it.path == '/caminho-dinamico/{param01}' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /caminho-dinamico/{param01}' } != null
assert endpoints.find { it.path == '/login' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /login' } != null
assert endpoints.find { it.path == '/_error/{code}.{_format}' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /_error/{code}.{_format}' } != null
assert endpoints.find { it.path == '/register' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /register' } != null
assert endpoints.find { it.path == '/caminho-dinamico/{param01}' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /caminho-dinamico/{param01}' } != null
assert endpoints.find { it.path == '/simple' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /simple' } != null
assert endpoints.find { it.path == '/simple_view' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /simple_view' } != null
assert endpoints.find { it.path == '/dynamic_route/{param01}/{param02}' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /dynamic_route/{param01}/{param02}' } != null
assert endpoints.find { it.path == '/error' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /error' } != null
assert endpoints.find { it.path == '/behind_auth' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /behind_auth' } != null
assert endpoints.find { it.path == '/telemetry' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /telemetry' } != null
assert endpoints.find { it.path == '/lucky/number' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /lucky/number' } != null
assert endpoints.find { it.path == '/lucky/fail' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /lucky/fail' } != null
assert endpoints.find { it.path == '/_error/{code}.{_format}' && it.method == 'GET' && it.operationName == 'http.request' && it.resourceName == 'GET /_error/{code}.{_format}' } != null

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Expect the UTF-8 route instead of the dev-only error route

When this endpoint-discovery check runs in the AppSec container, docker-init.sh sets APP_ENV=prod for both initialization and Apache requests, while config/routes/framework.yaml registers /_error/{code}.{_format} only under when@dev. The collected endpoints therefore cannot satisfy this assertion; the newly added /café/{item} route is the endpoint present in its place, so the assertion should reflect the production route set.

Useful? React with 👍 / 👎.

}
}
29 changes: 0 additions & 29 deletions appsec/tests/integration/src/test/www/symfony62/.env

This file was deleted.

10 changes: 0 additions & 10 deletions appsec/tests/integration/src/test/www/symfony62/.gitignore

This file was deleted.

31 changes: 0 additions & 31 deletions appsec/tests/integration/src/test/www/symfony62/Dockerfile

This file was deleted.

43 changes: 0 additions & 43 deletions appsec/tests/integration/src/test/www/symfony62/bin/console

This file was deleted.

85 changes: 0 additions & 85 deletions appsec/tests/integration/src/test/www/symfony62/composer.json

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

Loading
Loading