trafficserver icon indicating copy to clipboard operation
trafficserver copied to clipboard

ATS plugins are too tolerant of loading bad configs

Open traeak opened this issue 2 months ago • 1 comments

With ATS plugins like header rewrite the loaders themselves are very tolerant of errors inside the config files themselves, typically only blocking ATS starts and reloads when config files are missing. This is done by returning TS_ERROR from TSRemapNewInstance.

Example of header rewrite that should block reload:

# mistype of condition:
cond %{PSEUDO__REMAP_HOOK}

This might require further investigation of what might be required.

traeak avatar Nov 12 '25 19:11 traeak

@zwoop noticed this as well. This is an issue with some plugins not returning an error on init when they have bad config, even if they print an error.

We should probably identify the plugins that aren't returning errors and fix those. @zwoop might already have a list!

cmcfarlen avatar Nov 17 '25 23:11 cmcfarlen