View Issue Details

IDProjectCategoryView StatusLast Update
0000300Adventure PHP FrameworkBugpublic2016-05-11 09:14
ReporterChristianAchatzAssigned ToChristianAchatz 
PrioritynormalSeveritymajorReproducibilityalways
Status resolvedResolutionfixed 
Product Version3.2 
Target Version3.3Fixed in Version3.3 
Summary0000300: Application crushes with max execution time reached with missing space in tag definition
DescriptionIn case of tag definition

<fs:required-user-group>
   <fs:importdesign
attr1="..."
attr2="...">
      ...
   </fs:importdesign>
</fs:required-user-group>

application crashes with max execution time of 30s exceeded.
TagsNo tags attached.
Codereferenz: ([Datei]:[Zeile])
Namespacecore

Activities

ChristianAchatz

2016-05-10 16:32

administrator   ~0000715

Last edited: 2016-05-10 16:38

View 2 revisions

According to documentation, "\n" is not allowed in tag definitions. However, APF should not be running into exceeding execution time with improper tag definition but should be informing developers about the expectated definition.

ChristianAchatz

2016-05-11 09:13

administrator   ~0000716

- Added "\n" detection to throw exception instead of running into an endless loop.
- Fixed in release-3.2 and master(https://github.com/AdventurePHP/code/commit/6443e88d97ffa3b6ef173783714bda53cb42afe4).
- New version can be doanloaded via composer update, GitHub, or ZIP download.

ChristianAchatz

2016-05-11 09:14

administrator   ~0000717

Along with the fix, I've evaluated an extension to the parser detecting "\n" and using as tag delimiter the same way as " " (space). I'll follow up on that with release 3.3 development and evaluate whether an update to the parser is feasible in terms of performance.

Issue History

Date Modified Username Field Change
2016-05-10 16:30 ChristianAchatz New Issue
2016-05-10 16:30 ChristianAchatz Status new => assigned
2016-05-10 16:30 ChristianAchatz Assigned To => ChristianAchatz
2016-05-10 16:32 ChristianAchatz Note Added: 0000715
2016-05-10 16:38 ChristianAchatz Note Edited: 0000715 View Revisions
2016-05-11 09:13 ChristianAchatz Note Added: 0000716
2016-05-11 09:14 ChristianAchatz Note Added: 0000717
2016-05-11 09:14 ChristianAchatz Status assigned => resolved
2016-05-11 09:14 ChristianAchatz Fixed in Version => 3.3
2016-05-11 09:14 ChristianAchatz Resolution open => fixed