Prefer native Insert from URL over BE_HTTP_* by default#6
Open
fsans wants to merge 1 commit into
Open
Conversation
The native Insert from URL script step uses the same libcurl library as the BE_HTTP_* functions but is plugin-free and runs on all clients. Add a generic rule in SKILL.md and at the top of the http-urls reference: do not use BE_HTTP_* / BE_CurlSetOption to replace any functionality the native step can perform unless the user explicitly requests the plugin functions.
|
There's still some things that the BE functions can do that IFU can't. For example I was using BE HTTP GET to do a HEAD request to a server, combined with the CURLOPT_CERTINFO setting before the request, and then the BE_CurlGetInfo function after. I used that to pull out server certificate expiry info to know when I needed to replace a FMS certificate. So you may want to consider matching the curl options that BE allows with the ones that IFU does and comparing. ( If you did this and IFU caught up to BE then I'd happily deprecate the BE functions. ) And feel free to send me any PRs for modifications to documentation in the BE repo. Best of luck with this. Cheers, |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
SKILL.mddirecting the agent to prefer FileMaker's nativeInsert from URLscript step over theBE_HTTP_*/BE_CurlSetOptionplugin functions.BE_HTTP_*functions, requires no plugin, and runs on every client (FMP, FMS, Go, WebDirect).BE_HTTP_*/BE_CurlSetOptionto replace any functionality that the nativeInsert from URLscript step can perform, unless the user explicitly requests the plugin functions.references/http-urls.mdso it's visible whenever the HTTP reference is loaded.Test plan
baseelements-plugin/SKILL.mdand verify the new "HTTP — prefer native Insert from URL" pattern reads correctly under Critical Patterns, followed by the existing "HTTP (BE_HTTP_*) — configure before, inspect after" pattern.baseelements-plugin/references/http-urls.mdand verify the blockquote appears above theBE_HTTP_GETsection.