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

chore: Fix URL parameter concatenation #145

Merged
merged 3 commits into from
Feb 13, 2025
Merged

Conversation

joaorovera
Copy link
Contributor

Fixed URL concatenation to add '?' only if parameters are present

@RodrigoDornelles RodrigoDornelles changed the base branch from main to develop February 13, 2025 19:57
@RodrigoDornelles RodrigoDornelles merged commit 1cae849 into develop Feb 13, 2025
18 checks passed
@RodrigoDornelles RodrigoDornelles deleted the fixing-http_handler branch February 13, 2025 20:01
RodrigoDornelles added a commit that referenced this pull request Feb 25, 2025
* ci: docs workflow when modify samples or src

* docs: written a little more

* Update .gitattributes

* fix: mustache precedence msdos filter

* feat: support media on fengari driver html5

* merge (#144)

* tests: dropped luaunit (#140)

 - #138

* feat: samples in very small screens 80x24

* feat: asteroids in screen <= 400

* feat: improve responsive menu asteroids

* fix: self.params_dict is undefined #143

* fix: cleanup http native

* fix: std accoplament error in http

* style: fix luau linter

---------

Co-authored-by: Finalshare90 <[email protected]>

* feat: add support to fengari http module in html5

* chore: Fix URL parameter concatenation (#145)

* tests: dropped luaunit (#140)

 - #138

* chore: Fix URL parameter concatenation

---------

Co-authored-by: Finalshare90 <[email protected]>
Co-authored-by: João Vicente <[email protected]>
Co-authored-by: RodrigoDornelles <[email protected]>

* feat: add support to json in html5 fengari

* feat: start log system #14

* Add files via upload

* fix: main entry in npm packages

* fix: widescreen init html5

* ci: not allow non-ascii in codebase samples and src

* feat: media.music mock.music mock.video

* feat: add log system api to engine

* docs: improve

* docs: add example maze3d

* docs: maze3d wolf_newmap and reference

* change: read inteire file and split on build

* feat: add flag --telemedia190 to build command

* change: remove core repl from 'run' subcommand

* feat: sync Ginga font size with HTML

* fix: menu options order in asteroids

* fix: remove current relative path in util fs

* fix: custom dist on build

* fix: dont create empty file cli fs.move

* fix: mkdir in asset move

* fix: mkdir in asset move

* fix: luau-analyze warning media handler

* feat: get enverioment [WIP] [0.0.20]

* fix: icon32x32.bmp transparency in NDS

* docs: write about std.draw

* docs: add http in wget on README.md

* feat: http force protocol

* feat: optional fetch jsonrxi in fengari html5

* fix: transparenci in favicon.ico

* fix: add protect to inexistent images

* feat: add cache_set and cache_get

---------

Co-authored-by: Finalshare90 <[email protected]>
Co-authored-by: Joao Vicente <[email protected]>
Co-authored-by: João Vicente <[email protected]>
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