[Bug]: URL maximum length in E-Document Integration about alappextensions HOT 6 CLOSED

Please allow me to discuss internally! I will get back to you soon!

from alappextensions.

OK, needed to make sure I got that right. So if the plan is to increase the URL field from 250 to 2048, I approve! You can go ahead and make the change.
The solution you describe would be the right one! Given this is a log table, I could be tempted to ask for a breaking change exception, but it wouldn't be the "right" solution... the table is public and we hence must deprecate the old field, create a new one, copy the data, keep them in sync... Again given it's a log, you could consider if it's OK to truncate the URL and put it into the old field for backwards compatibility. By the book, the logs wouldn't support longer URLs until the old field is deprecated "removed" in 3 releases from now - but usually people cannot wait that long 😉
Anyway, looking forward to what you come up with!

from alappextensions.

Sounds fair to present a clear error message in this case. Approved.

from alappextensions.

As the logging happens after the request has been sent (to an external system) it seems to me too late to raise an error. Considering that I do not see a way to raise the error earlier in the pipeline without making significant changes to the current design, as the HttpRequest object is passed by reference into the (customized) interface implementation of the Send procedure. I assume this would need a more specific scope to move forward with a PR.

As we often face (REST-style) APIs with URLs longer than 250 characters (e.g. due to Guids as entity Ids, et cetera) it would make sense (in our teams opinion) to allow for longer URL paths as a more general solution. To expand the "E-Document Integration Log" URL it would be needed to implement a new field (e.g. "Request URL" or "API URL", change all references to the "old" URL field and mark the old field for pending obsoletion. Since this is obviously a more significant change, I will also not move forward without an approval for this scope.

If we get some guidance which direction (or even a simpler solution) should be implemented, we are happy to move forward with a PR in a timely manner.

from alappextensions.

We agree that a 250 char limit on URLs is sub-optimal and we would be open for improvements here 😊 As you know, the maximum length of a Text field in the database is 2048 characters. Would that suffice? Or are you looking for even longer strings?

from alappextensions.

As written in the original post (Additional context), our team believes that a limit of 2048 characters is an expected limit in the context of Business Central and, at best, no one will notice this limit in the future.

from alappextensions.