search

khengyun / autonext

automatic grading tool for edunext
34 stars 1 forks source link

EduNext API endpoints no longer need port! #25

Closed Kunniii closed 1 year ago

Kunniii commented 1 year ago

I just checked the extension this morning, the request to API server was not successful. This is because the API endpoint is no longer has a port, 8443 to be specific!

Please release a new version!! 🙌🙌🙌

Kunniii commented 1 year ago

Something stoopid is that, the /api/auth/token endpoints still using that 8443 port.

khengyun commented 1 year ago

Thank you for your assistance. I just rechecked and it appears that EduNext has removed the above port value. If a port value is not declared, APIs will use the default port values defined for each protocol type. For example, for HTTP and HTTPS, the default port values are 80 and 443, respectively. It's possible that they have decided to remove or change the port value on a weekly basis and unfortunately, I have not yet found a way to resolve this issue.

old: image

new: image

Kunniii commented 1 year ago

Looking forward to your fix, the extension is not working for now (at least, on my end)!!

await newRelease();
khengyun commented 1 year ago

I have updated the new patch, you can check it here: click

khengyun commented 1 year ago

Looking forward to your fix, the extension is not working for now (at least, on my end)!!

await newRelease();

you can check new release here: v3.5.2

Kunniii commented 1 year ago

Now the API uses port 8443, again, but auth endpoint doesn't! Nice play Edunext, i love you 3000

khengyun commented 1 year ago

Now the API uses port 8443, again, but auth endpoint doesn't! Nice play Edunext, i love you 3000

maybe i will update the server to automatically update the port for the api 🤦‍♂️

khengyun commented 1 year ago

@Kunniii i found solution for this problem https://github.com/khengyun/autonext/discussions/26#discussioncomment-5898519