Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

http_output results in libcurl error: URL using bad/illegal format or missing URL #2579

Closed
therealdwright opened this issue May 25, 2023 · 3 comments · Fixed by #2596
Closed
Labels

Comments

@therealdwright
Copy link
Contributor

Describe the bug

When configuring the Falco http_output URL as suggested in this reply
The URL is correctly assigned via environment variable, however when events are
generated from Falco I receive an error:
libcurl error: URL using bad/illegal format or missing URL

How to reproduce it

Install Falco using charts, configure outputs using the following method:
falco ... --option http_output.enabled=true --option http_output.user_agent="falcosecurity/falco" --option http_output.url="https://example.com/example/example2"

Expected behaviour

Output is posted to the endpoint

Screenshots
image

Environment

  • Falco version:
Falco version: 0.34.1
Libs version:  0.10.4
Plugin API:    2.0.0
Engine:        16
Driver:
  API version:    3.0.0
  Schema version: 2.0.0
  Default driver: 4.0.0+driver
  • System info:
{
  "machine": "x86_64",
  "nodename": "falco-m5jmx",
  "release": "5.4.241-150.347.amzn2.x86_64",
  "sysname": "Linux",
  "version": "#1 SMP Mon Apr 24 23:27:26 UTC 2023"
}
  • Cloud provider or hardware configuration:
    Amazon EC2, M5a series instance
  • OS:
NAME="Amazon Linux"
VERSION="2"
ID="amzn"
ID_LIKE="centos rhel fedora"
VERSION_ID="2"
PRETTY_NAME="Amazon Linux 2"
ANSI_COLOR="0;33"
CPE_NAME="cpe:2.3:o:amazon:amazon_linux:2"
HOME_URL="https://amazonlinux.com/"
  • Kernel:
    Linux 5.4.238-148.347.amzn2.x86_64 #1 SMP Thu Apr 6 19:42:57 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
  • Installation method:
    Kubernetes
@leogr
Copy link
Member

leogr commented May 25, 2023

cc @jasondellaluce @alacuku

@alacuku
Copy link
Member

alacuku commented May 29, 2023

Hi @therealdwright, the quotation marks around the URL are not handled correctly. It should work without them.

@therealdwright
Copy link
Contributor Author

the quotation marks around the URL are not handled correctly

Would you like me to push a PR to handle the quotes in the URL? I know you are busy with the next release milestone so would like to verify before spending time on writing any code.

therealdwright added a commit to therealdwright/falco that referenced this issue May 30, 2023
This commit will unescape URL's allowing them to be supported by
libcurl and eliminate any errors when a valid (quoted) URL is supplied
by a user.

Closes falcosecurity#2579

Signed-off-by: Daniel Wright danielwright@bitgo.com
therealdwright added a commit to therealdwright/falco that referenced this issue May 31, 2023
This commit will unquote URL's allowing them to be supported by
libcurl and eliminate any errors when a valid (quoted) URL is supplied
by a user.

Closes falcosecurity#2579

Signed-off-by: Daniel Wright danielwright@bitgo.com
therealdwright added a commit to therealdwright/falco that referenced this issue May 31, 2023
This commit will unquote URL's allowing them to be supported by
libcurl and eliminate any errors when a valid (quoted) URL is supplied
by a user.

Closes falcosecurity#2579

Signed-off-by: Daniel Wright danielwright@bitgo.com
therealdwright added a commit to therealdwright/falco that referenced this issue Jun 1, 2023
This commit will unquote URL's allowing them to be supported by
libcurl and eliminate any errors when a valid (quoted) URL is supplied
by a user.

Closes falcosecurity#2579

Signed-off-by: Daniel Wright danielwright@bitgo.com
poiana pushed a commit that referenced this issue Jun 5, 2023
This commit will unquote URL's allowing them to be supported by
libcurl and eliminate any errors when a valid (quoted) URL is supplied
by a user.

Closes #2579

Signed-off-by: Daniel Wright danielwright@bitgo.com
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants