Skip to content

HTTP Request helper#100

Closed
CodexisPhantom wants to merge 1 commit intoCommunityOx:mainfrom
CodexisPhantom:feat/http-request-helper
Closed

HTTP Request helper#100
CodexisPhantom wants to merge 1 commit intoCommunityOx:mainfrom
CodexisPhantom:feat/http-request-helper

Conversation

@CodexisPhantom
Copy link
Copy Markdown

This pull request introduces a new lib.httpRequest function to the library. This function provides a promise-based wrapper around FiveM's PerformHttpRequest native, allowing developers to perform HTTP requests with validation for common methods.

@ESK0
Copy link
Copy Markdown
Member

ESK0 commented Apr 20, 2026

@CodexisPhantom
Copy link
Copy Markdown
Author

This PR replaces 10 lines of code with a single-line wrapper. Like other ox_lib utilities, the goal is to eliminate redundant code across projects. Although it adds no new core functionality, the benefit is in the simplified developer experience and cleaner codebase.

@ESK0
Copy link
Copy Markdown
Member

ESK0 commented Apr 22, 2026

This PR replaces 10 lines of code with a single-line wrapper. Like other ox_lib utilities, the goal is to eliminate redundant code across projects. Although it adds no new core functionality, the benefit is in the simplified developer experience and cleaner codebase.

This PR adds 10 lines of code for a functionality that already exists as referenced in previous message.
PerformHttpRequestAwait

@CodexisPhantom
Copy link
Copy Markdown
Author

CodexisPhantom commented Apr 23, 2026

I missed the Await after PerformHttpRequest. I didn't see that native in the official docs, am I missing somewhere to look ?

@ESK0 ESK0 closed this Apr 23, 2026
@Maximus7474
Copy link
Copy Markdown
Member

I missed the Await after PerformHttpRequest. I didn't see that native in the official docs, am I missing somewhere to look ?

https://docs.fivem.net/docs/scripting-reference/runtimes/lua/functions/PerformHttpRequestAwait/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants