tools,examples,veb: fix v -d trace_before_request run examples/veb/veb_example.v after feedback from https://youtu.be/IuE6Bo1klK0?t=2100

Author: spytheman

cmd/tools/vtest-all.v

@@ -411,6 +411,11 @@ fn get_all_commands() []Command {
 			okmsg:  'A simple veb app, compiles with `-gc none -no-retry-compilation -cc tcc -d use_openssl` on macos and linux'
 			rmfile: 'examples/veb/todo/main'
 		}
+		res << Command{
+			line:   '${vexe} -d trace_before_request examples/veb/veb_example.v'
+			okmsg:  'examples/veb/veb_example.v compiles with `-d trace_before_request` on macos and linux'
+			rmfile: 'examples/veb/veb_example'
+		}
 	}
 	$if linux {
 		res << Command{

examples/veb/veb_example.v

@@ -19,9 +19,9 @@ struct Context {
 	veb.Context
 }
 
-pub fn (app &App) before_request() {
+pub fn (ctx &Context) before_request() {
 	$if trace_before_request ? {
-		eprintln('[veb] before_request: ${app.req.method} ${app.req.url}')
+		eprintln('[veb] before_request: ${ctx.req.method} ${ctx.req.url}')
 	}
 }
 

vlib/veb/middleware.v

@@ -282,10 +282,6 @@ pub fn decode_zstd[T]() MiddlewareOptions[T] {
 	}
 }
 
-interface HasBeforeRequest {
-	before_request()
-}
-
 pub const cors_safelisted_response_headers = [http.CommonHeader.cache_control, .content_language,
 	.content_length, .content_type, .expires, .last_modified, .pragma].map(it.str()).join(',')
 

vlib/veb/tests/veb_app_test.v

@@ -1,14 +1,20 @@
 // vtest build: present_sqlite3? // imports db.sqlite
 import veb
 import time
+import net.http
 import db.sqlite
 
 const port = 13004
 
 pub struct Context {
 	veb.Context
 pub mut:
-	user_id string
+	cid int
+}
+
+pub fn (mut ctx Context) before_request() {
+	dump(@LOCATION)
+	ctx.cid = 123
 }
 
 pub struct App {
@@ -18,6 +24,7 @@ pub mut:
 }
 
 pub fn (mut app App) before_accept_loop() {
+	dump(@LOCATION)
 	app.started <- true
 }
 
@@ -36,10 +43,12 @@ fn test_veb_application_compiles() {
 	spawn veb.run_at[App, Context](mut app, port: port, family: .ip, timeout_in_seconds: 2)
 	// app startup time
 	_ := <-app.started
-}
-
-pub fn (mut ctx Context) before_request() {
-	ctx.user_id = ctx.get_cookie('id') or { '0' }
+	res := http.fetch(url: 'http://127.0.0.1:${port}/get_cid')!
+	assert res.status_code == 200
+	assert res.body == '123'
+	okres := http.fetch(url: 'http://127.0.0.1:${port}/ok')!
+	assert okres.status_code == 200
+	assert okres.body == '{"success":true,"result":123}'
 }
 
 @['/new_article'; post]
@@ -103,7 +112,12 @@ fn (mut app App) some_helper[T](result T) ApiSuccessResponse[T] {
 
 // should compile, the route method itself is not generic
 fn (mut app App) ok(mut ctx Context) veb.Result {
-	return ctx.json(app.some_helper(123))
+	return ctx.json(app.some_helper(ctx.cid))
+}
+
+// should compile, the route method itself is not generic
+fn (mut app App) get_cid(mut ctx Context) veb.Result {
+	return ctx.text(ctx.cid.str())
 }
 
 struct ExampleStruct {

vlib/veb/veb.v

@@ -148,6 +148,11 @@ mut:
 	before_accept_loop()
 }
 
+interface HasBeforeRequestOnContext {
+mut:
+	before_request()
+}
+
 fn handle_route[A, X](mut app A, mut user_context X, url urllib.URL, host string, routes &map[string]Route) {
 	mut route := Route{}
 	mut middleware_has_sent_response := false
@@ -201,8 +206,8 @@ fn handle_route[A, X](mut app A, mut user_context X, url urllib.URL, host string
 	}
 
 	// first execute before_request
-	$if A is HasBeforeRequest {
-		app.before_request()
+	$if X is HasBeforeRequestOnContext {
+		user_context.before_request()
 	}
 	// user_context.before_request()
 	if user_context.Context.done {