Veb log middleware cross module printing

[Avey777]

logger_progect/admin/admin.v

module admin

import veb
import log

struct Context {
	veb.Context
}

pub struct Admin {}

@['/router'; get]
fn (app &Admin) index(mut ctx Context) veb.Result {
	log.info('${@METHOD}  ${@MOD}.${@FILE_LINE}')
	return ctx.json('success')
}

logger_progect/handler/app_start.v

module handler

import veb
import log
import admin { Admin }

const cors_origin = ['*', 'xx.com']

pub struct Context {
	veb.Context
}

pub struct App {
	veb.Middleware[Context]
	veb.Controller
	veb.StaticHandler
}

pub fn new_app() {
	log.info('${@METHOD}  ${@MOD}.${@FILE_LINE}')

	mut app := &App{}
	register_routes(mut app)

	app.use(handler: before_request)

	app.use(veb.cors[Context](veb.CorsOptions{
		origins:         cors_origin
		allowed_methods: [.get, .head, .patch, .put, .post, .delete, .options]
	}))

	port := 9009
	veb.run_at[App, Context](mut app, host: '', port: port, family: .ip6, timeout_in_seconds: 30) or {
		panic(err)
	}
}

@['/get'; get]
pub fn (app &App) index(mut ctx Context) veb.Result {
	// log.info('${@METHOD}  ${@MOD}.${@FILE_LINE}')

	return ctx.json('req success')
}

pub fn register_routes(mut app App) {
	app.register_controller[Admin, Context]('/admin', mut &Admin{}) or { log.error('${err}') }
}

pub fn before_request(mut ctx Context) bool {
	// $if trace_before_request ? {
	log.info('[veb] before_request: ${ctx.req.method} ${ctx.req.url}')
	// }
	return true
}

logger_progect/main.v

module main

import os
import log
import handler { new_app }

fn main() {
	mut l := log.Log{}
	l.set_output_stream(os.stdout())
	log.info('${@METHOD}  ${@MOD}.${@FILE_LINE}')

	new_app()
}

Request : http://127.0.0.1:9009/get
Output :[veb] before_request: GET /get

Request : http://127.0.0.1:9009/admin/router
No [veb] before_request: GET /admin/router output , Why?

source code： logger.zip

[medvednikov]

@Avey777 sorry, missed this discussion.

Has it been resolved?

[Avey777]

#24032

I discussed with my friend and believed that this should be a bug in veb. Now I bring up the issue

[Avey777]

@Avey777 sorry, missed this discussion.

Has it been resolved?

I think it should be possible to write code like this, or offer some other viable solution! This is important for custom middleware

[medvednikov]

Ah I see you created an issue.

[Avey777]

Ah I see you created an issue.

Yeah I think this should be a bug, what do you think?

I provided the source code, but I don't know why it doesn't work