recursivetree / seat-alliance-industry

Alliance industry marketplace for eve online
MIT License
2 stars 2 forks source link

Error 500 #10

Closed relance-haklar closed 1 year ago

relance-haklar commented 1 year ago

After updating to 1.1.8, whenever someone tries to supply an order they receive an error 500.

recursivetree commented 1 year ago

This does not happen for me. Can you send me what you order and the laravel logs of the error?

Here is the documentation of how to find the error log: https://eveseat.github.io/docs/troubleshooting/#checking-log-files

kaylak4520 commented 1 year ago

Hi @recursivetree,

I'm Mikayla. I work with @relance-haklar on our SeAT instance. We have many orders, which I will screenshot the list for you. (All have the same issue). When accessing any order and adding resources (IE Biomass: Quantity 1 and clicking add), we get a 500 internal server error. Screenshot 2023-01-05 at 12 59 09 PM (This is the orders list)

https:rxsk.seat.tempestcoalition.com:allianceindustry:order:331:deliveries:add.pdf (This is the app debug output after clicking "add" since installing 1.1.8)

laravel-2023-01-05.log Here is the log you asked for. Only showed in Web, not Worker docker.

recursivetree commented 1 year ago

Thanks for the logs, I'll take a look at it. It is completely normal that it only shows in the web container, because web handles all web related things(like handling a request). The worker container is mostly there to load data from the ESI.

recursivetree commented 1 year ago

On another note, did you ever install seat-inventory? The crash is related to seat-alliance-industry thinking seat-inventory is installed too, but then not actually finding it.

recursivetree commented 1 year ago

I found the issue. I'll release a fix shortly. In the meantime, you can install seat-inventory and the crash should go away.

kaylak4520 commented 1 year ago

THANK YOU SO MUCH!

I'll look into Seat-Inventory as well!

recursivetree commented 1 year ago

It should be fixed in the lastest release 1.1.9

kaylak4520 commented 1 year ago

@recursivetree I've confirmed that also installing recursivetree/seat-inventory fixes the issue. Thanks a bunch!

recursivetree commented 1 year ago

If you install plugins, you normally also update all other plugins, meaning you would have got the fix anyways.

Sorry for the inconvenience of this bug. But if people report bugs, it means the plugin is used, which is nice to hear.