search

thomasp85 / fiery

A flexible and lightweight web server
https://fiery.data-imaginist.com
Other
240 stars 12 forks source link

Add logging infrastructure #19

Closed thomasp85 closed 6 years ago

thomasp85 commented 6 years ago

Fixes #18

thomasp85 commented 6 years ago

Ok, I've made both the general logging format, as well as the access logging specifically, customisable through glue strings. It defaults to Common Log Format and there's a Combined Log Format string included as well - are we approaching what you expect @cpsievert (once routr errors gets relayed)?

codecov-io commented 6 years ago

Codecov Report

Merging #19 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #19    +/-   ##
======================================
  Coverage     100%   100%            
======================================
  Files           5      6     +1     
  Lines         548    651   +103     
======================================
+ Hits          548    651   +103
Impacted Files Coverage Δ
R/aaa.R 100% <ø> (ø) :arrow_up:
R/loggers.R 100% <100%> (ø)
R/Fire.R 100% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 8f5e154...a99d3fa. Read the comment docs.