Skip to content

[INTERNAL] Removes connectUi5Proxy documentation #722

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

Merged
d3xter666 merged 8 commits into from
Dec 23, 2022
Merged

[INTERNAL] Removes connectUi5Proxy documentation #722

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
9db0a0d
Removes connectUi5Proxy documentation
d3xter666 Dec 20, 2022
4241703
Document breaking change
d3xter666 Dec 20, 2022
2ab5acc
Add connectUi5Proxy to the migration guide of v3
d3xter666 Dec 20, 2022
043474c
Update docs/updates/migrate-v3.md
d3xter666 Dec 20, 2022
00e8d98
Update docs/pages/Configuration.md
d3xter666 Dec 20, 2022
4a379ac
Update migration guide
d3xter666 Dec 21, 2022
7e17277
Do not touch specVersions
d3xter666 Dec 21, 2022
05aa2a8
Clean up migration v3 guide
d3xter666 Dec 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 0 additions & 4 deletions docs/pages/Server.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ A project can also add custom middleware to the server by using the [Custom Serv
| `testRunner` | See chapter [testRunner](#testrunner) |
| `serveThemes` | See chapter [serveThemes](#servethemes) |
| `versionInfo` | See chapter [versionInfo](#versioninfo) |
| `connectUi5Proxy` | See chapter [connectUi5Proxy](#connectui5proxy) |
| `nonReadRequests` | See chapter [nonReadRequests](#nonreadrequests) |
| `serveIndex` | See chapter [serveIndex](#serveindex) |

Expand Down Expand Up @@ -64,9 +63,6 @@ Changes made to these `*.less` files while the server is running will automatica
### versionInfo
Generates and serves the version info file `/resources/sap-ui-version.json`, which is required for several framework functionalities.

### connectUi5Proxy
Provides basic proxy functionality using the proxy offered by [`connect-openui5`](https://github.com/SAP/connect-openui5#proxy) under the endpoint `/proxy`.

### nonReadRequests
Answers all non-read requests (POST, PUT, DELETE, etc.) that have not been answered by any other middleware with the 404 "Not Found" status code . This signals the client that these operations are not supported by the server.

Expand Down
12 changes: 12 additions & 0 deletions docs/updates/migrate-v3.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -126,3 +126,15 @@ The following processors have been removed:
^2^ Enabled for projects defining a [component preload configuration](../pages/Configuration.md#component-preload-generation)
^3^ Enabled in `self-contained` build, which disables `generateComponentPreload` and `generateLibraryPreload`
^4^ Enabled for projects defining a [bundle configuration](../pages/Configuration.md#custom-bundling)

## Removal of Standard Middleware

The following middlewares have been removed from the [standard middlewares list](../../pages/Server/#standard-middleware):
Comment thread
flovogt marked this conversation as resolved.
Comment thread
flovogt marked this conversation as resolved.

- connectUi5Proxy

**Middleware Migration**

| UI5 Tooling v2 | UI5 Tooling v3 | Note |
| --------------------------- | --------------------------- | ------------------------- |
| connectUi5Proxy | *None* | More sophisticated proxy solutions for ui5-server are already available in the form of [custom middleware extensions from the UI5-community](https://bestofui5.org/#/packages?tokens=proxy:tag). Please refactor any custom middleware that attaches to `beforeMiddleware` or `afterMiddleware` of `connectUi5Proxy` to reference some other middleware. |
Comment thread
flovogt marked this conversation as resolved.