Make the request logging middleware optional

[duffn]

Make this optional with a configuration setting:

grape-api-boilerplate/app/root.rb

Lines 13 to 19 in 080c8cb

	# Logging
	insert_before Grape::Middleware::Error, GrapeLogging::Middleware::RequestLogger,
	{
	logger:,
	formatter: GrapeLogging::Formatters::Json.new,
	include: [GrapeLogging::Loggers::FilterParameters.new([:password])]
	}

[Aaryanpal]

Hi @duffn ,
Could you please assign so I can resolve this issue. Can u please be little discriptive about this issue and how can I replicate to resolve

[duffn]

Thanks!

There's no issue really, but only an enhancement.

If you run the project per the README instructions and make a sample request to curl http://localhost:3000/api/v1/hello, you'll see that you get a request log line like

grape-api-boilerplate-app-1  | {"date":"2023-03-18T22:56:13.095+00:00","severity":"INFO","data":{"status":200,"time":{"total":0.74,"db":0,"view":0.74},"method":"GET","path":"/api/v1/hello","params":{},"host":"localhost"}}

This is setup here:

grape-api-boilerplate/app/root.rb

Lines 13 to 19 in 080c8cb

	# Logging
	insert_before Grape::Middleware::Error, GrapeLogging::Middleware::RequestLogger,
	{
	logger:,
	formatter: GrapeLogging::Formatters::Json.new,
	include: [GrapeLogging::Loggers::FilterParameters.new([:password])]
	}

I think adding this middleware should be optional. Some users may not want it or it may not be necessary.

The issue is to make this optional using a configuration setting: https://github.com/duffn/grape-api-boilerplate/tree/9e3e423473e273fcedf293ecb02109be59886bd8/config/settings