Skip to content

Cleaner default error page - #1146

Closed
vlyon wants to merge 1 commit into
PerlDancer:masterfrom
vlyon:pr/cleaner-default-error
Closed

vlyon wants to merge 1 commit into
PerlDancer:masterfrom
vlyon:pr/cleaner-default-error

Conversation

@vlyon

@vlyon vlyon commented Mar 30, 2016

Copy link
Copy Markdown
Contributor
  • Cleaner, clearer display of the context of an error. Uses table instead of pre for formatting.
  • Now shows 5 lines of context instead of almost 3. (Bug fixed)
  • Renamed backtrace to error_context since it returns the context of the error not a backtrace.

@vlyon vlyon mentioned this pull request Mar 30, 2016
@xsawyerx

Copy link
Copy Markdown
Member

👍 without the commit for renaming.

@veryrusty

Copy link
Copy Markdown
Member

👍

@xsawyerx

Copy link
Copy Markdown
Member

@vlyon Is the renaming commit required? The reason I would prefer without it is because it has nothing to do with the CSS cleanup and it's just semantics. I'd like to have smaller PRs if possible.

@vlyon

vlyon commented Mar 31, 2016

Copy link
Copy Markdown
Contributor Author

@xsawyerx The rename is not needed. The only reason for it, is because backtrace is badly named. It doesn't produce a stack-trace it just displays the error context, the stack-trace is produced by get_caller.

Shall I remove the rename?

@veryrusty

Copy link
Copy Markdown
Member

Merged as 71012a4 (including undoing the name change. Twas simpler to JFDI).

Thanks @vlyon !

@veryrusty veryrusty closed this Apr 1, 2016
@xsawyerx

xsawyerx commented Apr 2, 2016

Copy link
Copy Markdown
Member

@vlyon Thank you for the contribution and for the patience!

xsawyerx added a commit that referenced this pull request Apr 19, 2016
    [ BUG FIXES ]
    * GH #1102: Handle multiple '..' in file path utilities.
      (Oleg A. Mamontov, Peter Mottram)
    * GH #1114: Fix missing prereqs as reported by CPANTS.
      (Mohammad S Anwar)
    * GH #1128: Shh warning if optional megasplat is not present.
      (David Precious)
    * GH #1139: Fix incorrect Content-Length header added by AutoPage
      handler (Michael Kröll, Russell Jenkins)
    * GH #1144: Change tt tags to span in skel (Jason Lewis)
    * GH #1046: "no_server_tokens" configuration option doesn't work.
      (Sawyer X)
    # GH #1155, #1157: Fix megasplat value splitting when there are empty
      trailing path segments. (Tatsuhiko Miyagawa, Russell Jenkins)
      NOTE: Paths matching a megasplat that end with a '/' will now include
      an empty string as the last value. For the route pattern '/foo/**',
      the path '/foo/bar', the megasplat gives ['bar'], whereas '/foo/bar/'
      now gives ['bar','']. Joining the array of megasplat values will now
      always be the string matched against for the megasplit.

    [ DOCUMENTATION ]
    * GH #1119: Improve the deployment documentation. (Andrew Beverley)
    * GH #1123: Document import of utf8 pragma. (Victor Adam)
    * GH #1132: Fix spelling mistakes in POD (Gregor Herrmann)
    * GH #1134: Fix spelling errors detected by codespell (James McCoy)
    * GH #1153: Fix POD rendering error. (Sawyer X)

    [ ENHANCEMENTS ]
    * GH #1129: engine.logger.* hooks are called around logging a message.
      (Russell @veryrusty Jenkins)
    * GH #1146: Cleaner display of error context (Vernon Lyon)
    * GH #1085: Add consistent keywords for accessing headers;
      'request_header' for request, 'response_header', 'response_headers'
      and 'push_response_header' for response. (Russell @veryrusty Jenkins)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants