[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
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.

Sign up for GitHub

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

Jump to bottom

API failure when uploading rockspec #1660

Open
mrcjkb opened this issue Apr 4, 2024 · 4 comments
Open

API failure when uploading rockspec #1660

mrcjkb opened this issue Apr 4, 2024 · 4 comments
Milestone
LuaRocks 4

Comments

@mrcjkb
Copy link
Contributor
mrcjkb commented Apr 4, 2024

Please read the following before submitting:

Please fill out the following:

Platform:

  • Linux

LuaRocks version:

  • 3.11.0

LuaRocks output from when the issue occurred:

Sending toml-edit-0.3.3-1.rockspec ...
fs.which_tool("downloader")
fs.is_tool_available("curl", "curl")
fs.tmpname()
fs.download("https://luarocks.org/api/1/***/check_rockspec?package=toml%2dedit&version=0%2e3%2e3%2d1", "/tmp/lua_v588fp")
fs.use_downloader("https://luarocks.org/api/1/***/check_rockspec?package=toml%2dedit&version=0%2e3%2e3%2d1", "/tmp/lua_v588fp", nil)
fs.absolute_name("/tmp/lua_v588fp")
fs.current_dir()
fs.which_tool("downloader")
fs.execute_quiet("wget --no-check-certificate --no-cache --user-agent=\"LuaRocks/3.9.2 linux-x86_64 via wget\" --quiet --timeout=30 --tries=1  --output-document ", "/tmp/lua_v588fp", "https://luarocks.org/api/1/***/check_rockspec?package=toml%2dedit&version=0%2e3%2e3%2d1")
fs.Q("/tmp/lua_v588fp")
fs.Q("https://luarocks.org/api/1/***/check_rockspec?package=toml%2dedit&version=0%2e3%2e3%2d1")
fs.execute_string("wget --no-check-certificate --no-cache --user-agent=\"LuaRocks/3.9.2 linux-x86_64 via wget\" --quiet --timeout=30 --tries=1  --output-document  '/tmp/lua_v588fp' 'https://luarocks.org/api/1/***/check_rockspec?package=toml%2dedit&version=0%2e3%2e3%2d1'")
os.execute: 	wget --no-check-certificate --no-cache --user-agent="LuaRocks/3.9.2 linux-x86_64 via wget" --quiet --timeout=30 --tries=1  --output-document  '/tmp/lua_v588fp' 'https://luarocks.org/api/1/<redacted>/check_rockspec?package=toml%2dedit&version=0%2e3%2e3%2d1'
Results: 1
  1 (number): 1536
fs.change_dir_to_root()
stderr: 
Error: API failure: failed downloading https://luarocks.org/api/1/***/check_rockspec?package=toml%2dedit&version=0%2e3%2e3%2d1 - /check_rockspec?package=toml%2dedit&version=0%2e3%2e3%2d1

Full build log: https://github.com/vhyrro/toml-edit.lua/actions/runs/8558149732/job/23452158770

Description:

This only affects the toml-edit rock. Other workflows work fine.
@hishamhm
Copy link
Member
hishamhm commented Apr 5, 2024

@mrcjkb weird — does it fail consistently? could it be an API key misconfiguration, perhaps?

@teto
Copy link
teto commented Apr 6, 2024

API key misconfiguration should resolve to a 401 ? can we see the full output of wget ?

@vhyrro
Copy link
vhyrro commented Apr 7, 2024

Indeed it seems like it was some sort of API key issue! I did consider that initially but the error was too weird for me to believe it was something wrong with the token.

Would be awesome if luarocks could do some sort of preliminary check before trying to access the API just in case :D

@hishamhm
Copy link
Member
hishamhm commented Apr 8, 2024

LuaRocks does respond the HTTP code when not using wget (for example, when running the LuaRocks binaries). That behavior will become the default in LuaRocks 4.

@hishamhm hishamhm added this to the LuaRocks 4 milestone Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
LuaRocks 4
Development

No branches or pull requests
4 participants
@hishamhm @teto @mrcjkb @vhyrro