Re: libswoc test failures (10.x build with assertions enabled)

2025-01-09 Thread Walt Karas
I see this in your trace: > /builddir/build/BUILD/trafficserver-10.0.2-build/trafficserver-10.0.2/lib/swoc/include/swoc/Errata.h:1002 Looking at: https://github.com/apache/trafficserver-libswoc/blob/SolidWallOfCode-patch-1/code/include/swoc/Errata.h The file is under 1000 lines.  Perhaps you ar

client_allow_list plugin

2024-08-17 Thread Walt Karas
In 2021, Yahoo tried to open source this plugin.  But the effort fizzled:   https://github.com/apache/trafficserver/pull/7598 . The plugin now has preliminary support for client screening using an HTTP header that an Amazon Application Load Balancer can be configured to add to incoming requests.

Re: [E] Proposal: move ts_util.h/.cc into TS API (in ts namespace)

2024-04-18 Thread Walt Karas
Hmm ts_util is a utility currently only used by txn_box. It uses libswoc, as do other parts of our code. We have already decided to bring ts_util into our repo. We already have to maintain it. It's a question of whether we want to get more use out of it. If we don't think it's of significant v

Re: [E] Re: Proposal: move ts_util.h/.cc into TS API (in ts namespace)

2024-04-18 Thread Walt Karas
wrapper as an independent > library. > > On Wed, Apr 17, 2024 at 7:15 PM Walt Karas > wrote: > > > The atscppapi was deprecated because it was buggy and had bad > performance. > > > > On Wed, Apr 17, 2024 at 4:36 PM Masakazu Kitajo > wrote: > > > >

Re: [E] Re: Proposal: move ts_util.h/.cc into TS API (in ts namespace)

2024-04-17 Thread Walt Karas
many questions to throw +1 for this. Since the wrapper is just a > wrapper, we can see if it's good without having it as TS API. Do we have a > reason to skip doing that? > > On Wed, Apr 17, 2024 at 1:06 PM Walt Karas > wrote: > > > The atscppapi had problems, but I don&#

Re: [E] Re: Proposal: move ts_util.h/.cc into TS API (in ts namespace)

2024-04-17 Thread Walt Karas
e of libswoc, etc.). One might > make a better wrapper. I don't see a reason to have such things as TS API > and maintain it by ourselves. IMO, TS API should provide minimal things > which can be only done by API. > > > On Wed, Apr 10, 2024 at 2:50 PM Walt Karas > wrote: &g

Re: [E] Re: Proposal: move ts_util.h/.cc into TS API (in ts namespace)

2024-04-10 Thread Walt Karas
C > APIs in C++? > > — Leif > > On Mon, Apr 8, 2024 at 13:28 Walt Karas > wrote: > > > That is move > plugins/experimental/txn_box/plugin/include/txn_box/ts_util.h > > to include/ts, and plugins/experimental/txn_box/plugin/src/ts_util.cc to > > src/api . > > &

Proposal: move ts_util.h/.cc into TS API (in ts namespace)

2024-04-08 Thread Walt Karas
That is move plugins/experimental/txn_box/plugin/include/txn_box/ts_util.h to include/ts, and plugins/experimental/txn_box/plugin/src/ts_util.cc to src/api . May involve some picking and choosing. ts_util.h includes common.h. I don't think we want to bring all of common.h into the TS API.

Re: [E] [PROPOSAL] New Get() API for the Txn error body

2024-03-14 Thread Walt Karas
Why not more const safe? const char *TSHttpTxnErrorBodyGet(TSHttpTxn txnp, size_t *buflength, const char **mimetype); On Thu, Mar 14, 2024 at 5:09 PM Leif Hedstrom wrote: > We have an existing API to set the error body (TSHttpTxnErrorBodyGet() ), > and I’d like to add the corresponding Get’er f

Re: [E] Ats_problem

2024-01-03 Thread Walt Karas
Has anyone written an Au test where the actual body size does not match the size given in the Content-Length header field? If not, that would be a good addition. On Wed, Jan 3, 2024 at 11:25 AM Walt Karas wrote: > Have you considered removing cached objects with PURGE requests (sent w

Re: [E] Ats_problem

2024-01-03 Thread Walt Karas
Have you considered removing cached objects with PURGE requests (sent with curl)? https://docs.trafficserver.apache.org/admin-guide/storage/index.en.html#removing-an-object-from-the-cache On Wed, Jan 3, 2024 at 9:13 AM amirhossein ramezan wrote: > Dear Sir/madam > I just had two questions for yo

Re: [E] Re: [PROPOSAL] Remove "proxy.process" prefix from all core metrics

2023-12-18 Thread Walt Karas
Is the current naming based on some SNMP MIB spec? Personally, my indifference is boundless. This change will I think lead to some logistic aggravation, due to scripts that take metrics output as input. Any chance it's worth it to make the prefix configurable? On Mon, Dec 18, 2023 at 9:41 PM Ja

Re: [E] Request for Guidance on Debugging a Plugin for Apache Traffic Server

2023-12-13 Thread Walt Karas
Have you read this part of the documentation yet? https://docs.trafficserver.apache.org/developer-guide/debugging/index.en.html Other sections of the documentation cover building ATS from source and installing it. Autotools are used for the current version, ATS 9, and cmake is used for the versio

Re: [E] [TSAPI] TSTxnAlloc

2023-12-12 Thread Walt Karas
anism) to do the cleanup on TXN_CLOSE.\ > I don't see how your answers are related to my questions so I'll let others consider this. > -Original Message- > From: Walt Karas > Sent: Thursday, December 7, 2023 9:27 AM > To: dev@trafficserver.apache.org > Subject: Re: [E] [TSAPI] TSTxnAlloc > > >

Re: [E] [TSAPI] TSTxnAlloc

2023-12-07 Thread Walt Karas
ot ATS resources. > > -Original Message- > From: Walt Karas > Sent: Wednesday, December 6, 2023 9:25 PM > To: dev@trafficserver.apache.org > Subject: Re: [E] [TSAPI] TSTxnAlloc > > Could we move Arena.h to tscpp/util, and just add: > > Arena & tsapi::txnArena(T

Re: [E] [TSAPI] TSTxnAlloc

2023-12-06 Thread Walt Karas
Could we move Arena.h to tscpp/util, and just add: Arena & tsapi::txnArena(TSHttpTxn txn); For your finalizer example, when would that be better than using Cleanup.h? That is, when would it be better than having the vector be a data member of the TxnAuxData class you provide as a parameter to th

Updated invitation: ATS 10 Development Meeting @ Every 2 weeks from 11am to 12pm on Tuesday (EST) (dev@trafficserver.apache.org)

2023-11-29 Thread Walt Karas
...@yahooinc.com ATTENDEE;CUTYPE=INDIVIDUAL;ROLE=REQ-PARTICIPANT;PARTSTAT=ACCEPTED;RSVP=TRUE ;CN=Walt Karas;X-NUM-GUESTS=0:mailto:wka...@yahooinc.com ATTENDEE;CUTYPE=INDIVIDUAL;ROLE=REQ-PARTICIPANT;PARTSTAT=ACCEPTED;RSVP=TRUE ;CN=Brian Neradt;X-NUM-GUESTS=0:mailto:brian.ner...@yahooinc.com ATTENDEE

Updated invitation: ATS 10 Development Meeting @ Every 2 weeks from 11am to 12pm on Tuesday from Tue Aug 29 to Tue Dec 5 (EDT) (dev@trafficserver.apache.org)

2023-11-29 Thread Walt Karas
-GUESTS=0:mailto:damian.me...@yahooinc.com ATTENDEE;CUTYPE=INDIVIDUAL;ROLE=REQ-PARTICIPANT;PARTSTAT=TENTATIVE;RSVP=TRU E;CN=Walt Karas;X-NUM-GUESTS=0:mailto:wka...@yahooinc.com ATTENDEE;CUTYPE=INDIVIDUAL;ROLE=REQ-PARTICIPANT;PARTSTAT=ACCEPTED;RSVP=TRUE ;CN=Brian Neradt;X-NUM-GUESTS=0:mailto:brian.ner

Re: [E] RE: What should we do with noncopyable.h?

2023-11-06 Thread Walt Karas
er.apache.org > Subject: Re: What should we do with noncopyable.h? > > I'd say get rid of it (when we delete the C++ API). I don't like having > that kind of general utility in our tree. > > Masakazu > > On Fri, Oct 27, 2023 at 7:20 AM Walt Karas > wrote: >

What should we do with noncopyable.h?

2023-10-27 Thread Walt Karas
https://github.com/apache/trafficserver/blob/master/include/tscpp/api/noncopyable.h Given that the C++ API is deprecated in 10 and will be deleted in 11. Should we get rid of noncopyable (since it's less useful in C++11 and later)? Or, move it to tscpp/util ?

Re: [E] Re: Vote: Remove autotools build files from the repository (ATS 10)

2023-10-20 Thread Walt Karas
I checked Jason's Yahoo internal release build of ATS, and it does put the path to the Magick libs in the plugin rpaths. So, apparently that's my bad, I'm missing some configure option or double secret environment variable. On Fri, Oct 20, 2023 at 10:22 AM Walt Karas wrote: >

Re: [E] Re: Vote: Remove autotools build files from the repository (ATS 10)

2023-10-20 Thread Walt Karas
Using ldd, I get this list of libraries that my installed developer build of ATS depends on. That is, the libraries used by the installed executables and libraries, including all core plugins, experimental or not. libbrotlicommon.so.1 => /opt/edge/brotli/1.0/lib/libbrotlicommon.so.1 libbrotlienc.

Re: [E] Re: Vote: Remove autotools build files from the repository (ATS 10)

2023-10-19 Thread Walt Karas
I think each company should successfully build their internal version with cmake before autotools goes away. I don't think Yahoo has done this yet. On Thu, Oct 19, 2023 at 11:55 AM Brian Neradt wrote: > +1 > > Maintaining both build tools would be very burdensome. > > On Thu, Oct 19, 2023 at 10

Re: memory leak issues

2023-09-15 Thread Walt Karas
I'd strongly recommend putting the relevant autotools config.nice (or a hopefully-exisitng cmake equivalent) in the issue. Has information like whether you used gcc or clang. On Fri, Sep 15, 2023 at 10:56 AM Walt Karas wrote: > When you write an issue for a memory leak, please make

memory leak issues

2023-09-15 Thread Walt Karas
When you write an issue for a memory leak, please make it clear whether you found it building (assuming autotools) with the configure option --enable-asan, or --enable-lsan . This may be important in reproducing the issue.

Re: [E] Re: problem compiling

2023-09-11 Thread Walt Karas
On the master branch, cstdint is already included: https://github.com/apache/trafficserver/blob/master/lib/swoc/include/swoc/TextView.h On Mon, Sep 11, 2023 at 10:30 AM jean-frederic clere wrote: > On 9/11/23 15:59, jean-frederic clere wrote: > > On 9/11/23 15:40, jean-frederic clere wrote: > >>

Re: [E] Re: Proposal: TS API: Don' t require a DbgCtl instance of have a tag when constructed.

2023-09-08 Thread Walt Karas
iod != Cript::string::npos) { plugin_debug_tag = plugin_debug_tag.substr(slash + 1, period - slash - 1); } dbg_ctl_cript = TSDbgCtlCreate(plugin_debug_tag.c_str()); On Thu, Sep 7, 2023 at 10:21 PM James Peach wrote: > > > On 6 Sep 2023, at 10:30 am, Walt Karas > wrote: > &g

Re: [E] Re: Proposed new API: TSVConnFdGet

2023-09-07 Thread Walt Karas
+1 as long as I don't have to do it. On Wed, Sep 6, 2023 at 10:39 PM Brian Neradt wrote: > +1 > > We already have a TSVConnFdCreate, so it seems appropriate to add a getter. > > On Wed, Sep 6, 2023 at 5:26 PM SUSAN HINRICHS > wrote: > > > I would like to add a convenience API to fetch the file

Proposal: TS API: Don' t require a DbgCtl instance of have a tag when constructed.

2023-09-05 Thread Walt Karas
See https://github.com/apache/trafficserver/pull/10375 .

Re: [E] Proposal: make utilities declared in Cleanup.h part of TS API

2023-09-05 Thread Walt Karas
ail secure email. > > --- Original Message --- > On Tuesday, September 5th, 2023 at 11:37 AM, Walt Karas > wrote: > > > > Would it be possible to propose concrete alternatives? We can't just > > simply delete Cleanup.h, the xdebug plugin won't c

Re: [E] Proposal: make utilities declared in Cleanup.h part of TS API

2023-09-05 Thread Walt Karas
on Mail secure email. > > --- Original Message --- > On Tuesday, September 5th, 2023 at 10:53 AM, Masakazu Kitajo < > m4s...@gmail.com> wrote: > > > > I was not talking about the need to use RAII. I just don't think we > should > > promote Cleanup.h.

Re: [E] Proposal: make utilities declared in Cleanup.h part of TS API

2023-09-05 Thread Walt Karas
It seems that some are not convinced of the need to use RAII. I won' t get into that, it's easy to find writeups advocating for it, which are better than anything I could write. I have no strong feelings about how things are spelled or abbreviated. On Sat, Sep 2, 2023 at 11:22 PM James Peach wr

Re: [E] Re: Proposal: make utilities declared in Cleanup.h part of TS API

2023-08-31 Thread Walt Karas
behavior change. Then ours would be > useless. It's not only about "helmet". Any utilities would have this issue. > This is why I'm skeptical to have utilities as TS API and asking all if we > want to do this. > > On Thu, Aug 31, 2023 at 1:35 PM Walt Karas > wro

Re: [E] Re: Proposal: make utilities declared in Cleanup.h part of TS API

2023-08-31 Thread Walt Karas
ary after we make it better. > > Not discarding Cleanup.h and promoting it to TS API are different things. > TS API is not something we can casually change or remove. I'd rather not > add a short lifetime temporary solution to TS API. > > > > On Wed, Aug 30, 2023 at 12:19 

Re: [E] Re: Proposal: make utilities declared in Cleanup.h part of TS API

2023-08-30 Thread Walt Karas
API? > > -- Masakazu > > On Tue, Aug 22, 2023 at 11:48 AM Walt Karas > wrote: > > > See the PR > https://urldefense.com/v3/__https://github.com/apache/trafficserver/pull/10231__;!!Op6eflyXZCqGR5I!HHlP-H_IgX9Ah46LlOiHqX--VcnzF3Dp5uG2JI_JXBmG8c_jMQNpsYQPvyDaaAsympzb7neltiG

Proposal: make utilities declared in Cleanup.h part of TS API

2023-08-22 Thread Walt Karas
See the PR https://github.com/apache/trafficserver/pull/10231 . Cleanup.h is currently available to plugins in include/tscpp/api . The proposal is to move it to include/ts . The declarations currently in the atscppapi namespace are moved to the tsapi::c_support namespace. The c_support sub-name

Re: [E] [VOTE] Removing example/plugins/cpp-api

2023-08-17 Thread Walt Karas
/apache/trafficserver/pull/10231 On Wed, Aug 16, 2023 at 6:18 PM Walt Karas wrote: > +1 > > On Wed, Aug 16, 2023 at 2:39 PM Chris McFarlen wrote: > >> The cpp plugin API is being deprecated for 10.0 (tscpp/api/*). I'd like >> to just remove the examples immediately

Re: [E] [VOTE] Removing example/plugins/cpp-api

2023-08-16 Thread Walt Karas
+1 On Wed, Aug 16, 2023 at 2:39 PM Chris McFarlen wrote: > The cpp plugin API is being deprecated for 10.0 (tscpp/api/*). I'd like to > just remove the examples immediately for 10.0 since we don't need examples > for a deprecated API. Removing them now will also create less work for the > CMake

Re: [E] How to log total server response time

2023-06-01 Thread Walt Karas
something wrong… > > > > PS: I didn’t found any issue on github about it. > > > > Thanks again > > Nuno Baeta > > > > *From:* Walt Karas > *Sent:* 1 de junho de 2023 16:06 > *To:* Nuno Jose Baeta > *Subject:* Fwd: [E] How to log total server resp

Re: [E] [PROPOSAL] New API to look up strings against our set of WKS constants

2023-05-14 Thread Walt Karas
As a background question, is there a trie somewhere that the HTTP header parser (in the core) uses to return WKS pointers for WKSs in the headers? I'm having trouble seeing how the speedup of comparing to WKS pointers is not negated by the time needed to detect WKSs in HTTP headers. On Sun, May 14

Re: [E] Proposal for TS API in 9.2.1: Remove TSSslSecretXxx functions

2023-04-27 Thread Walt Karas
Yes, but Yahoo won't be using that plugin with open source 9.2.x . On Thu, Apr 27, 2023 at 7:03 PM James Peach wrote: > > > > On 28 Apr 2023, at 5:16 am, Walt Karas > wrote: > > > > It's like there are some cases where these functions do work. But >

Re: [E] Proposal for TS API in 9.2.1: Remove TSSslSecretXxx functions

2023-04-27 Thread Walt Karas
It's like there are some cases where these functions do work. But noone's responded and said they've successfully used them. I'd hope anyone writing plugins is subscribed to this mailing list. On Thu, Apr 27, 2023 at 12:17 AM James Peach wrote: > > > > On 27

Re: [E] Re: Proposal for TS API in 9.2.1: Remove TSSslSecretXxx functions

2023-04-26 Thread Walt Karas
Everything I know is in this issue: https://github.com/apache/trafficserver/issues/9562 ? Are you saying you don't understand it? Can you be more specific about what isn't clear? On Wed, Apr 26, 2023 at 9:48 PM James Peach wrote: > > > > On 25 Apr 2023, at 1:01 p

Re: Proposal for TS API in 9.2.1: Remove TSSslSecretXxx functions

2023-04-26 Thread Walt Karas
Speak now or forever hold you peace. On Mon, Apr 24, 2023 at 10:01 PM Walt Karas wrote: > https://github.com/apache/trafficserver/pull/9587 indicates they do not > work. There are no definite plans to fix them. So, seems they are > unlikely to be fixed before the release of 9.2.1.

Proposal for TS API in 9.2.1: Remove TSSslSecretXxx functions

2023-04-24 Thread Walt Karas
https://github.com/apache/trafficserver/pull/9587 indicates they do not work. There are no definite plans to fix them. So, seems they are unlikely to be fixed before the release of 9.2.1. Fixing or removing them in 10.0 is a related but separate issue.

Re: [E] Feedback on removing AIO_MODE_NATIVE from iocore/aio

2023-04-03 Thread Walt Karas
I believe there's a concern about io_uring in that using it means ATS can now crash the kernel, which is harder to debug than a process crash. On Mon, Apr 3, 2023 at 12:38 PM Chris McFarlen wrote: > I would like to remove this disk IO mode from ATS. If nobody is using it, > then I'd like to dele

Re: [E] Re: Proposed change to TS API: TSHeapBuf

2023-01-04 Thread Walt Karas
x27;d deprecate the old API and remove it in the future. > That's a lot of work. > > I feel like introducing TSVarLenData has more downside than the upside. > > On Wed, Jan 4, 2023 at 9:13 AM Walt Karas > wrote: > > > They could make the returned structure a nesting stru

Re: [E] Proposed change to TS API: TSHeapBuf

2023-01-04 Thread Walt Karas
> > > On Jan 4, 2023, at 3:03 PM, Walt Karas > wrote: > > > > There are people at Yahoo who have written plugins in straight C, and > don't > > want to change to C++. > > They can continue to write plugins in C, just have to compile with a C++ > compile

Re: [E] Re: Proposed change to TS API: TSHeapBuf

2023-01-04 Thread Walt Karas
g plug-ins with C but require C++? Then we can introduce nice > things in ts.h. The old C plugins just have to be changed to compile with > C++, and ABI compatibility within major version must be enforced. > > — Leif > > > On Sep 6, 2022, at 10:24, Walt Karas > wrote: >

Re: [E] Re: Proposed change to TS API: TSHeapBuf

2023-01-04 Thread Walt Karas
ght want > to store the returned pointer into a user-defined structure. > > > > On Tue, Sep 6, 2022 at 10:24 AM Walt Karas > wrote: > > > Presumably we don't need to allow for use of antiquated C compilers that > > don't allow structures as return valu

Proposed new TS API function: TSDbgCtlDestroy()

2022-12-21 Thread Walt Karas
void TSDbgCtlDestroy(TSDbgCtl const *dbg_ctl); Destroys a TSDbgCtl created by a call to TSDbgCtlCreate(). This is needed so that the data structure containing all of the TSDbgCtl instances is not destroyed until all references to it have been destroyed. (The references are counted.)

Proposed change to TS C++ API

2022-11-09 Thread Walt Karas
A small chage is proposed in this PR: https://github.com/apache/trafficserver/pull/9187 In Cleanup.h, allow the continuation of TxnAuxDataMgr to be used for general purposes.

Proposed new TS API function: TSContWhenCreate()

2022-10-14 Thread Walt Karas
TSCont TSContWhenCreate(TSEventFunc funcp, TSMutex mutexp); With this change, TSCont objects will have an implicit "retry" flag. The retry flag will be true when the TSCont is created with TSContWhenCreate(). The flag will be false when the TSCont is created with TSContCreate(). If the retry fl

plugin debug trace output

2022-10-05 Thread Walt Karas
FYI, this is a way to make your (non-core) plugin build with the new (much lower overhead) debug interface, or the old one if the the new one is not available in the ATS version you need the .so to run with: #if defined(TSDbg) const TSDbgCtl *dbg_ctl_yada = TSDbgCtlCreate("yada"); #define DBG_YAD

Re: [E] Re: Proposed change to TS API: TSHeapBuf

2022-09-06 Thread Walt Karas
nges, the fix and the new > API, should be made like 1 + 1, but not 1.5 + 0.5 nor 1 + 0.5 + 0.5. > > Masakazu > > On Thu, Sep 1, 2022 at 10:08 AM Walt Karas > wrote: > > > As a rule of thumb, I prefer using encapsulation/data abstraction. I > think > > perhaps

Re: [E] Re: Proposed change to TS API: TSHeapBuf

2022-08-31 Thread Walt Karas
ug 31, 2022 at 8:36 AM Masakazu Kitajo wrote: > > > > What's the advantage of using TSHeapBuf? What issue does it solve? > > > > > > On Wed, Aug 31, 2022 at 7:48 AM Walt Karas > > wrote: > > > > > Described here: > > > > > > &

Re: [E] Re: Proposed change to CPP API

2022-08-31 Thread Walt Karas
Wow, your question is actually answered in docs: https://docs.trafficserver.apache.org/developer-guide/introduction/header-file-structure.en.html?highlight=cppapi . Run out and buy a lottery ticket, the goddess of good fortune is smiling upon you today. On Wed, Aug 31, 2022 at 7:44 PM Walt Karas

Re: [E] Re: Proposed change to CPP API

2022-08-31 Thread Walt Karas
wrote: > Can someone tell me what should be under tscpp/api and tscpp/util ? > > On Wed, Aug 31, 2022 at 5:07 AM Walt Karas > wrote: > > > I propose moving Cleanup.h from plugins/xdebug to include/ts/api as a > part > > of > https://urldefense.com/v3/__https://git

Proposed change to TS API: TSHeapBuf

2022-08-30 Thread Walt Karas
Described here: https://github.com/apache/trafficserver/blob/os_pkey_cnf_reload/doc/developer-guide/api/functions/TSHeapBuf.en.rst#tsheapbufdata , In PR https://github.com/apache/trafficserver/pull/8790 . This allows a dynamically allocated buffer, of any reasonable length, to be returned by a TS

Proposed change to CPP API

2022-08-30 Thread Walt Karas
I propose moving Cleanup.h from plugins/xdebug to include/ts/api as a part of https://github.com/apache/trafficserver/pull/9044 .

Re: [E] Fwd: Lua plugin development - Global Variables

2022-07-13 Thread Walt Karas
There is a way to do this in the C API (global arguments with TSUserArgSet/Get). But it doesn't look like these functions are supported in the Lua API. Can you keep the data in a file, perhaps on a RAM disk to make access fast? On Wed, Jul 13, 2022 at 4:41 AM Vishal Garg wrote: > Hey, > ATS Ve

Re: [E] failed to add remap rule - FQ pacing plugin

2022-06-23 Thread Walt Karas
What's at line 200 in your remap.config file? On Thu, Jun 23, 2022 at 2:25 PM Vasanth Sekkar wrote: > Hi, > > I am facing issue after enabling FQ pacing plugin in remap.config. I am > seeing the following error log in diags.log. Can you please guide me on > this? > > [Jun 23 18:53:01.590] traffi

Re: [E] Re: [VOTE] new API for getting the SNI for a client connection

2021-10-06 Thread Walt Karas
If we really want to swing for the bleachers, we could move all config into a plugin. In a way that would allow a proxy to be managed with SNMP, gRPC, etc. by replacing the plugin. On Tue, Oct 5, 2021 at 12:33 PM Robert O Butts wrote: > +1 > > IMO we should have APIs for all SSL/SNI data. > > O

Re: [E] Proposal to update existing TS API : TSHttpTxnAborted

2021-06-01 Thread Walt Karas
Oh I see, for cases where there is no status? On Tue, Jun 1, 2021 at 10:13 AM Walt Karas wrote: > Hmm, you can't use TSHttpHdrStatusGet() ? > > On Sat, May 29, 2021 at 9:19 AM Sanjay Singh > wrote: > >> Hi all, >> >> I'd like to propose a chan

Re: [E] Proposal to update existing TS API : TSHttpTxnAborted

2021-06-01 Thread Walt Karas
Hmm, you can't use TSHttpHdrStatusGet() ? On Sat, May 29, 2021 at 9:19 AM Sanjay Singh wrote: > Hi all, > > I'd like to propose a change to an existing TS API, "TSHttpTxnAborted". > > The current function signature: > TSReturnCode > TSHttpTxnAborted(TSHttpTxn txnp) > > Propsed change: > TSReturn

Re: [E] Re: Is there a performance issue with --enable-debug binaries?

2021-04-20 Thread Walt Karas
--enable-debug means that the code will be compiled without the -O3 option, that is to say, no optimization. It's not related to whether trace/debug output is configured on or off. You can enable trace/debug output even if you did not specify --enable-debug when you run configure. On Tue, Apr 20

Re: [E] RE: Help with plugin loading in ATS 7.1.x vs ATS 9.0.x.

2021-03-19 Thread Walt Karas
If I remember correctly, I don't think, in ATS9, you can load the same plugin in both plugin.config and remap.config. But I'm not sure that's a new restriction in 9 versus 7. On Fri, Mar 19, 2021 at 9:19 PM Alan Carroll wrote: > Yes, sorry, I'm OOO this week. You can disable it in "records.conf

Proposed performance improvement of tagged debug output

2021-02-08 Thread Walt Karas
https://github.com/apache/trafficserver/pull/7452 This PR changes TS API. It introduces this new type: typedef struct TSFDbgCtl_s { char volatile on; // Flag char const *tag; } TSFDbgCtl; And this function: tsapi TSFDbgCtl const *TSFDbgCtlCreate(char const *tag); And this new printf-like

Re: [E] Re: [DISCUSS] ATS code study groups

2021-02-02 Thread Walt Karas
The other option would be to write tutorials. If it's easier to do talks, hopefully they would be recorded. We could scrape together money for a tech writer and get written tutorials from the videos. On Tue, Feb 2, 2021 at 4:35 PM Evan Zelkowitz wrote: > Can put me down as interested in all of

Re: Proposed changes to regex_remap core plugin

2020-11-20 Thread Walt Karas
e" is not used). "pristine" will mean the pristine URL is used in place of the remapped, for matching as well as substitution. On Fri, Nov 13, 2020 at 12:54 PM Walt Karas wrote: > 1. Two-character string substitutions can begin with ^ as well as $. If > ^ is used, and

Proposed changes to regex_remap core plugin

2020-11-13 Thread Walt Karas
1. Two-character string substitutions can begin with ^ as well as $. If ^ is used, and the substitution is a URL component, the component will be the one in the pristine, rather than the remapped, URL. So, for example, $h is the host in the remapped URL, and ^h is the host in the pristine URL.

Re: ATS performance degradation when debug output is enabled

2020-11-02 Thread Walt Karas
Note also that the PR included an implementation of read/write locking that avoids writer starvation, which apparently is an issue with std::shared_mutex and pthreads rwlock. On Mon, Nov 2, 2020 at 10:20 AM Walt Karas wrote: > Since this was mentioned during the summit, I just wanted

ATS performance degradation when debug output is enabled

2020-11-02 Thread Walt Karas
Since this was mentioned during the summit, I just wanted to highlight that there was an issue opened for this, I created a PR, but it was never merged: https://github.com/apache/trafficserver/issues/6150

Re: [E] Re: Proposed new TS plugin API function: TSHttpHdrSchemeGet

2020-10-07 Thread Walt Karas
> On Tue, Oct 6, 2020 at 3:22 PM Walt Karas .invalid> > wrote: > > > So, when I call the current TSUrlSchemeGet() for these two requests: > > > > printf "GET / HTTP/1.1\r\nHost: mYhOsT.teSt:61000\r\n\r\n" | nc localhost > > 61001 > > printf

Re: [E] Proposal for new TS plugin API function: TSUrlPortGet

2020-10-06 Thread Walt Karas
lRawPortGet that returns 0 > if the port is not explicitly present in the URL. > > On Tue, Sep 29, 2020 at 9:28 AM Walt Karas > wrote: > >> This gets the port string in the URL. If the port is not in the URL, it >> returns nullptr (and sets length to zeor). Note this is

Re: [E] Re: Proposed new TS plugin API function: TSHttpHdrSchemeGet

2020-10-06 Thread Walt Karas
edstrom wrote: > >> > > >> > The point here being to make a new API that replaces the old, without > >> breaking compatibility? And this new API has special semantics on a > cache > >> hit vs cache miss? > >> > > >> > This seems

Re: [E] [API] - TSMgmtDataTypeGet - Proposal

2020-09-30 Thread Walt Karas
> +1 > > > > No, that's basically it. Except I see no reason this information > shouldn't > > be accessible. > > > > On Mon, Sep 28, 2020 at 7:06 PM Walt Karas > .invalid> > > wrote: > > > > > Yes, that makes sense, if you are readin

Proposal for new TS plugin API function: TSUrlPortGet

2020-09-29 Thread Walt Karas
This gets the port string in the URL. If the port is not in the URL, it returns nullptr (and sets length to zeor). Note this is subtly different from TSUrlPortGet which always returns a valid port, the canonical port for the scheme if the port is not specified. Currently there is no way to detect t

Proposed new TS plugin API function: TSHttpHdrSchemeGet

2020-09-28 Thread Walt Karas
This should get the scheme for the request. This differs from `TSUrlSchemeGet` in that it gets the scheme even if it is not in the URL of the request. For most proxy requests, the ATS core will remove the host and scheme in the request while tracking it internally. In such a case a plugin cannot di

Re: [E] [API] - TSMgmtDataTypeGet - Proposal

2020-09-28 Thread Walt Karas
> type and call the appropriated Get (TSMgmtIntGet, TSMgmtStringGet, etc). > > As someone who implemented something similar but for the internal > RPC(jsonrpc) this function could be handy. > > > Thanks, > Damian. > > On Mon, Sep 28, 2020 at 10:02 PM Walt Karas >

Re: [E] [API] - TSMgmtDataTypeGet - Proposal

2020-09-28 Thread Walt Karas
guration variable name is not > known at compile time. > > On Mon, Sep 28, 2020 at 3:46 PM Walt Karas .invalid> > wrote: > > > So, more concretely, is it for future safety? For example, if the value > is > > a percentage, and it's currently an int, but you suspect it

Re: [E] [API] - TSMgmtDataTypeGet - Proposal

2020-09-28 Thread Walt Karas
whether > to call TSMgmtIntGet or *TSMgmtFloatGet*. There is currently no way to know > programmatically. The code has to "just know" which it is. > > On Mon, Sep 28, 2020 at 2:10 PM Walt Karas .invalid> > wrote: > > > Can you describe an example situation where

Re: [E] [API] - TSMgmtDataTypeGet - Proposal

2020-09-28 Thread Walt Karas
Can you describe an example situation where this would be useful? On Mon, Sep 28, 2020 at 1:59 PM Damian Meden wrote: > Hi Guys. > > I would like to propose adding a new API function to bring the record data > type from a particular record. > > A brief description of this new proposed API: > > >

Re: [E] Re: [PROPOSAL] TS API for Status,Note,Warning,Alert

2020-09-09 Thread Walt Karas
+1 On Wed, Sep 9, 2020 at 11:57 AM Bryan Call wrote: > +1 > > -Bryan > > > > On Sep 8, 2020, at 10:04 AM, Aaron Canary > wrote: > > > > I'd like to propose adding API calls for the remaining methods in diag.h: > > TSStatus(const char *fmt, ...) // prints to diags log (informational) > > TSNote(

Re: [E] TS API Review: New HTTP/2 stream id and priority getters

2020-08-31 Thread Walt Karas
Would traffic replay use this information? More generally, what I'm wondering is, how does traffic dump coordinate with TS's logging capability? I think typically, the TS API provides these sorts of handles to plugins so they can change the processing of the transaction/session/etc. We could als

Re: [E] Proposal: extend our pre-commit hook to run autopep8

2020-07-31 Thread Walt Karas
+1 On Thu, Jul 30, 2020 at 6:29 PM Brian Neradt wrote: > Hi dev@trafficserver.apache.org, > > We currently run clang-format as a part of our pre-commit hook. This is a > helpful tool for applying our formatting standards on our C++ files. I > suggest we extend the hook to also inspect any Python

Re: [PROPOSAL] GitHub change to Squash and Merge

2020-06-15 Thread Walt Karas
+1 as long as it doesn't cause uncommented commits. On Mon, Jun 15, 2020 at 9:43 AM Steven R. Feltner wrote: > +1 > > On 6/14/20, 11:32 AM, "Bryan Call" wrote: > > Notice: This email is from an external sender. > > > > I would like to propose that we switch to "Squash and Merge" for >

Re: CPP API change - TXN_CLOSE event handling in TransactionPlugin

2020-05-18 Thread Walt Karas
Hmm I think if you forgot to call 'Transaction::addPlugin()' you would just leak the plugin object, I don't think it would cause the crash you saw. On Mon, May 18, 2020 at 6:38 PM Sudheer Vinukonda wrote: > We recently ran into a core dump issue due to one of our > TransactionPlugin's continuati

Re: Throttler utility

2020-04-03 Thread Walt Karas
make HostDB timing/updates > considerably more readable. > > On Tue, Mar 31, 2020 at 4:58 PM Walt Karas > wrote: > >> Should we add a utility like this to include/ts/util ? >> >> class Throttle >> { >> public: >> Throttle(unsigned secondsBetween =

Throttler utility

2020-03-31 Thread Walt Karas
Should we add a utility like this to include/ts/util ? class Throttle { public: Throttle(unsigned secondsBetween = 60); // Returns true the first time it's called, and if the time since the last time it returned true is // more than secondsBetween seconds. // bool operator () (); //

Re: Proposal for utility to avoid use of stale pointers to objects that have been destroyed.

2020-03-27 Thread Walt Karas
Here is some (simplistic) example code using NoStale: https://github.com/ywkaras/MiscRepo/blob/master/NoStale/example.cc On Thu, Mar 26, 2020 at 11:53 AM Walt Karas wrote: > https://github.com/ywkaras/MiscRepo/blob/master/NoStale/NoStale.h >

Re: Thread affinity on Continuations vs TSContSchedule API

2020-03-26 Thread Walt Karas
I was having problems with this too in a new plugin I was writing. I had to make this small change to get it to work: https://github.com/apache/trafficserver/pull/6489/files#diff-8c09c39a3b13e183ba1bfd1e590cfb21 I found that I had to use TSContScheduleOnPool() instead of TSContSchedule(). On Thu

Proposal for utility to avoid use of stale pointers to objects that have been destroyed.

2020-03-26 Thread Walt Karas
https://github.com/ywkaras/MiscRepo/blob/master/NoStale/NoStale.h

null_transform.c

2020-03-06 Thread Walt Karas
Some questions about null_transform.c (the mother of all transform plugins). Is this, https://github.com/apache/trafficserver/blob/d9f647ca7c15d0eb4687fa59706ab246f49ece1b/example/plugins/c-api/null_transform/null_transform.c#L192 , really a reliable way to make sure all transform-related resource

Re: [API Proposal] Global Arg slots

2020-03-04 Thread Walt Karas
12:31 PM Leif Hedstrom wrote: > > > > > > > > On Mar 4, 2020, at 11:17 AM, Walt Karas .INVALID> > > wrote: > > > > > > So why do you like this? > > > > - Less code > > - Fewer APIs > > - Generalizes the concept > > >

Re: [API Proposal] Global Arg slots

2020-03-04 Thread Walt Karas
So why do you like this? - Breaks lots of existing code. - Less typesafe. - Makes use more verbose. It's good how the API has avoided the use of void pointers. We should not reverse that design goal. On Wed, Mar 4, 2020 at 12:03 PM Leif Hedstrom wrote: > > > > On Feb 27, 2020, at 10:13 AM, Le

Re: [API Proposal] Global Arg slots

2020-02-27 Thread Walt Karas
I'm thinking perhaps we should instead pass a const pointer to: typedef struct { TSReturnCode (*globalArgIndexReserve)(const char * name, const char * description, int * arg_idx); TSReturnCode (*globalArgIndexNameLookup)(const char * name, int * arg__idx, const char ** description); TSRetur

Re: [API Proposal] Global Arg slots

2020-02-27 Thread Walt Karas
Did you mean: void TSGlobalArgSet(int arg_idx, void * arg) void * TSGlobalArgGet(int arg_idx) ??? On Thu, Feb 27, 2020 at 11:13 AM Leif Hedstrom wrote: > Hi all, > > This is an idea from Bryan Call, which he suggested as a solution for > reloadable plugins changes that are now in 9.0.x. I re

Re: [API Proposal] Global Arg slots

2020-02-27 Thread Walt Karas
Sudheer, can you create a branch in your trafficserver fork, that switches an existing plugin from using TxnArg to GlobalArg? I don't see how that would work typically. I believe the point is that you want a different value for the same arg index for each transaction. On Thu, Feb 27, 2020 at 11:

Re: Async providers and std::future

2020-01-21 Thread Walt Karas
Using async seems roughly equivalent to scheduling a Continuation. Calling it will likely create a thread or a thread pool, on top of the ones in the ATS core. atscppapi also has a C++ wrapper for TSCont (in Continuation.h). I tried to get in some improvements to this ( https://github.com/apache

  1   2   3   >