Register resources for cleanup

Add the possibility to close transport session and the ability
to register resource to be closed.

For this we also add onTransportChannelClosed() into
TransportChannelListener which is invoked when channel is closed.

JIRA: NETCONF-714
Change-Id: I4912bd7d4a03530e4941bc9b663b2dc3b54a2f22
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Signed-off-by: Samuel Schneider <samuel.schneider@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Force instanceNum to be a long

We require this parameter and will issue a 500 internal server error if
it is not present.

Push the definition to be a plain long, so as to guide JAX-RS (and
future users) towards non-Strings.

JIRA: NETCONF-1370
Change-Id: Ia01b09721536a1a343ab27beff093fb8f107856a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move NO_CONTENT and NOT_FOUND response

These responses are rather useful, make them more widely available.

JIRA: NETCONF-1379
Change-Id: I35c76c89afeb05458337a268d1833524a2e89cc5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

HTTPServerSession.prepare() is version-agnostic

HttpVersion is only required for HttpResponse construction, i.e. we have
it passed to Response.toHttpResponse(HttpVersion). Remove them from
prepation path.

Change-Id: I0aeadf8a5e97ef9ffe4d98ab6a1405173a18bc83
JIRA: NETCONF-1379
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Mark outbound write thread is internal

When we are writing out a message, we may be hitting a KEX -- at which
point we would normally be blocked.

The problem is that the blocked thread is usually the Netty IO thread,
which effectively ends up deadlocking it.

Prevent this situation by marking our thread as internal, which causes
SSHD to enqueue the message rather than blocking.

JIRA: NETCONF-1415
Change-Id: If472986270e827a316099168069754fe1661939d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix javadocs in Subscriber(s)

Fix typos and wrong links in Subscriber and Subscribers javadocs.
Make clear that we operate on RestconfStream.

Change-Id: I16b3388a3e5bdd74048f1c7993f3d0dbc0b46ad8
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Bump upstreams

Adopt:
- mdsal-14.0.4
- controller-10.0.4
- aaa-0.20.3

Change-Id: I4317628af4a0b24004919d3faee45f955c423ca2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix docs in RestconfStream

We are no more using web-sockets but SSE.
Fix documentation accordingly.

Change-Id: I656e5c77071413479c2ce62e83113f0bf62ae584
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Fix docs in SSESender

We are no more using web-sockets but SSE.
Fix documentation accordingly.

Change-Id: Id1aab876a7c63ee2d31e3890775645c6888e9e46
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Add range restriction for backoff-multiplier

Add range restriction from 1 to max value. This will remove
the case when it starts decreasing instead of multiplying the value
initially set as the minimum value.

JIRA: NETCONF-1407
Change-Id: I2bb906386b9935543e8a4b6eab23e90313d3f4f2
Signed-off-by: Peter Suna <peter.suna@pantheon.tech>

Clean up XRD output

Use writeEmptyElement() to ensure we have a self-closed tag.

JIRA: NETCONF-1379
Change-Id: I2112e42b3cfccbe683bd9f618f9fd072825b5410
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Share a single DOMSchemaService across devices

Since we have the same model context, we can just use a
FixedDOMSchemaService, which does the magic.

This reduces the amount of code we have as well as eliminate a single
object per instance, improving scalability a bit.

Change-Id: I28e0f468647b54b49623065eabd816b326474dd9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add FIXME to MdsalOperationService

JIRA: NETCONF-1414
Change-Id: I9e1587bafabb5e93d340e8e079e95223df764101
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Introduce transport.spi

We have a number of implementation-internal classes living in
transport.api. Create transport.spi, so that implementation details are
in a different package.

Change-Id: I4701c632dc41f8202214068495711c640acd49a4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Handle the case of a null private key

A null object is an indication that there are no more objects left --
handle that gracefully.

A null private key can cause an exception during the creation of a
PEMParser instance. A check has been added to ensure that the
private key is not null.

JIRA: NETCONF-1411
Change-Id: Iebe4da892bfcc518882403b838899060b4deea6b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Yaroslav Lastivka <yaroslav.lastivka@pantheon.tech>

Clarify description in odl-netconf-device

Update description for max-backoff-millis and min-backoff-millis
leaf to clarify their functionality.

JIRA: NETCONF-1408
Change-Id: Iccefcad853c49303402fda21c01748cb21172539
Signed-off-by: Peter Suna <peter.suna@pantheon.tech>

OpenApi: Fix /mounts endpoint

After removal of JacksonJaxbJsonProvider usage there is no writer
capable to write List of MountPointInstance(s) to JSON.

Create dedicated MountPointsEntity to generate OpenApiEntity.

JIRA: NETCONF-1412
Change-Id: I6f645deacc51841d00d0fd2db574c26163ca18f2
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Modernize keystore-legacy

Use DataObjectIdentifier instead of InstanceIdentifier.

Change-Id: I030c8dfc89fa3d1707c8db54aa8d7777ee12ff06
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Address NETCONF-1210 FIXME

NETCONF-1210 has been fixed thus we can un-comment assertion.

Change-Id: I5c98363a6e5fd0a5c3cd97d2f8abddf1a89de54e
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

OpenAPI: remove empty list/container from payloads

Adjusted depth logic to remove empty lists and containers.
Now logic should work in line with proposed in NETCONF-1298.
Removed POST for depth 1 - for that value child container/list
always result in empty payload because child elements are being
cut off by depth.

JIRA: NETCONF-1345
Change-Id: I4f83823ac06d63dc57c69963f0eb8f6a1e3bb671
Signed-off-by: Oleksandr Zharov <oleksandr.zharov@pantheon.tech>

Fix client dispatcher queuing

Future listeners are not delivered immediately upon write completion,
but rather at some time after.

This means we need to setup the queue before we send the request, so
that if we receive a response before the callback runs we do not get
into weird state.

JIRA: NETCONF-1406
Change-Id: I66e1911fd6db6843499f772827e7ae00fa3ff377
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move AbstractJukeboxTest

We have contracts in restconf.server.api which could use some localized
testing. Allow that to happen by rehosting AbstractJukeboxTest.

Change-Id: Ic31f667a3bee4f661f2f1b5555b25e24c7e72c7d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Adopt draft-ietf-netconf-restconf-client-server-38

Pick up latest upgrades to the draft.

Change-Id: Ife09d7ba979f52ad9e812b587c0af521a31498c0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate TestUtils

This contains only a single mis-named method and a number of static
nested classes.

This patch:
- converts nested classes to top-level, reducing the need for imprts
- moves freePort() to AbtractE2ETest.randomBindablePort() with some
  documentation
- introduces transport.http.Http{Client,Server}StackConfiguration
  for wide reuse.

Change-Id: I7f269551654761545406e591c7e7562ea5aacf70
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Allocate TCPClient earlier

Mirror the layout of TCPServer, where we allocate the handle beforehand
-- which implies a validity check before we attempt to do anything.

Change-Id: Ib26512a3f36e4156a5c33ecccf2c7f8486648f78
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use Empty.immediateFuture()

yang.common.Empty has a utility method, use that instead of our own
constant.

Change-Id: I6e2f59367c066467ad6df8d6ba285633e810157a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Introduce transport.api.NettyTransportSupport

Rehost most of the logic from transport-tcp to transport-api, so it can
be used by both transport-tcp and transport-udp.

Change-Id: Ie52a6365a9aae28f9509eeab6be260db3ac97be8
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Expand NettyImpl with datagram support

Add basics needed to support datagram channels. This will be useful for
transport-udp.

Change-Id: I290f7677cf581fc66ccffbaf50dfc942388f1e12
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Disconnect AbstractNettyImpl from ietf-tcp-common

The Netty implementation should not have direct bindings to a concrete
model, so it can be reused between transport-{tcp,udp}.

Change-Id: I447163b7db2d2c1bfccae9042c661e3c2d999bfb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix NettyTransportSupport documentation

We have a copy&paste error in javadoc links, fix that up.

Change-Id: Ib0171370c7806d5012b7788b294e7bb38fec06e4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

End-to-end tests for Netty RESTCONF server

Integration tests provided for Netty based RESTCONF server
implementation. Instance of Netty endpoint with basic
controller models is created then HTTP requests are performed
using Netty based client from transport-http.

Tests covering requests to /data (incl actions and yang-patch),
/operations, /yang-library-version, /modules URIs plus
/.well-known/host-meta{.json} URIs.

Additional scenarios:
- topology node mount using device simulator
- data change notifications event stream
- device notification event stream

JIRA: NETCONF-1367
Change-Id: Ibb9dc960cc03b1a7215cb8ff39e2fd2bd136db0e
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Signed-off-by: Oleksandr Zharov <oleksandr.zharov@pantheon.tech>
Signed-off-by: Matej Sramcik <matej.sramcik@pantheon.tech>
Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>
Signed-off-by: Yaroslav Lastivka <yaroslav.lastivka@pantheon.tech>

Clean up ClientFactoryManager configuration

We have split configuration processing -- the key is acquired only after
the connection is established. Refactor this by acquiring the bits
needed and then perform minimal changes to ClientFactoryManager.

This also adds a FIXME for a follow up, which would not require access
to SSH implementation internals.

JIRA: NETCONF-1190
Change-Id: I223597f3b2cfc7f5e898e830874d3e142eb10acc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert transport-ssh to a JPMS module

Now that we are not hosting models, we can become a full JPMS module. As
a temporary side-effect, also specify Automatic-Module-Name in
shaded-sshd.

JIRA: NETCONF-1190
Change-Id: Ia19cec291d368dcfe75a97716003ba633685e10b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add encoding to AbstractPendingRequest

Fix picking encoding inside AbstractPendingRequest#onFailure
which causes NPE during the initialization of FormattableDataResponse.

JIRA: NETCONF-1379
Change-Id: I336a28b20f08402f79636dc8a97c4e15d1282d01
Signed-off-by: Yaroslav Lastivka <yaroslav.lastivka@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Ensure backoffMillis is not less than expected

Keeping the last backoff value after randomization can
reduce the SleepingTask timeout to a much smaller value
than expected.

Fix this by keeping the backoff value before randomization.

JIRA: NETCONF-1408
Change-Id: Ie926c92f9e203515c9623d9c794e441a447962f7
Signed-off-by: Peter Suna <peter.suna@pantheon.tech>

Adopt draft-ietf-netconf-udp-client-server-04

Import ietf-udp-{client,server}.yang as found in the latest version of
draft-ietf-netconf-udp-client-server -- in transport-udp.

Change-Id: Id483a3f195346ebea51530be35cd6cb0d4c321e9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Adopt draft-ietf-netconf-netconf-client-server-37

Upgrade models to match current draft, at revision 2024-08-14.

Change-Id: Iee29808e943abf4945d7d11876f9ce57af5e420d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix odl-netconf-transport dependencies

Make sure we use a range for odl-mdsal-model-rfc7317-iana-crypt-hash.

Change-Id: I555ea90c0c840ee3ee107687d4e32336028571ea
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 325167b5cb205456cbc6a5fe8fd84606c67d1c1c)

Convert transport-tls to a JPMS module

Now that we do not have generated code here, we can switch to being a
JPMS module, properly encapsulating our internals.

Change-Id: Ib8c06203408f53b40db996b868329ae928dd1745
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Switch to using RFC9645 models

ietf-tls-{client,common,server}.yang have been published in RFC9645 and
are available from MD-SAL.

Change-Id: If36b3c6070223fc81c34354bbcc6c4ea96289be2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Allow status code 201

The POST request returns 201 when resource is created successfully,
so it should not generate WARN log when the test tool sees the response
code. Include it in the list of accepted status codes.

JIRA: NETCONF-1410
Change-Id: I6b1d5e099bba285ecf6f1653ef9ce188eb12bd8c
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Refactor SessionListener integration

Disconnect SSHTransportStack from SessionListener, so the implementation
detail is hidden from users. An additional side-effect is that we no
longer need to suppress SpotBugs.

JIRA: NETCONF-1190
Change-Id: I79c28aba49064289dfa7ce903522b1da60482004
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert transport-tcp to a JPMS module

We have moved models out, convert now to a JPMS module to improve Java
integration.

Change-Id: I3eb8603af7a66f1f102d9a126bbdad492ee334cc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Switch to using RFC9644 models

ietf-ssh-{client,common,server}.yang have been published in RFC9644 and
are available from MD-SAL.

This patch switches to using them instead of the old draft, adapting to
the new way how algorithms are modeled.

Change-Id: Iaac864c7ec315794c92f6bd8313a580a9d4b75ec
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Switch to using RFC9643 models

ietf-tcp-{client,common,server}.yang have been published in RFC9643 and
are available from MD-SAL.

This patch switches to using them instead of the old draft, but only
supports single address binding for now.

It also requires updating the client/server models to match the
multi-listen capability -- which we pick up in the exact shape as the
latests drafts.

Change-Id: Id2ee382b6d988679a65a2812b51af8653e22d6c1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use RFC9642 ietf-keystore.yang

This model has been finalized and published and is available from
MD-SAL. Pick it up from there, leaving keystore-api a vacuous artifact
for now.

Change-Id: Ia99d3d1947fb343586d32f8ae6e420af5ffd6588
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use RFC9641 ietf-truststore.yang

This model has been finalized and published and is available from
MD-SAL. Pick it up from there, leaving truststore-api a vacuous artifact
for now.

Change-Id: I105a25b91ed2f06acec60295f1af716f7e69285d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use RFC9640 ietf-crypto-types

MD-SAL is providing ietf-crypto-types.yang, hence we do not have to
package it ourselves anymore.

Change-Id: I066a1055f78882ba8c5ab26639f9654da0d4f95a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump upstreams

Adopt:
- odlparent-14.0.4
- infrautils-7.0.4
- yangtools-14.0.5
- mdsal-14.0.3
- controller-10.0.3
- aaa-0.20.2

Change-Id: Ide732539861eda8d123f66a2c5849f326f4ed2f6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fixup checkstyle

Upgraded checstyle is catching a lot of javadoc violations, this patch
fixes them up.

Change-Id: Ib156cb891e748b957cf0dd9415d043c35c62d01c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add RFC6415 constructs

The XRD and related contracts should be modeled, so that we can actually
look them up.

This creates LinkRelation and WellKnownURI constant registries and
models the XRD, at least the very bare bones we require.

We then proceed to define responses and use them in WellKnownResources.

JIRA: NETCONF-1379
Change-Id: Id24fe3dc9d1798440277286d182506feabafdc55
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix JAX-RS JRD output

'links' is an array, not a singleton -- correct that.

Change-Id: I10ef1929d1ec13a5a111e7063baeca62141892e8
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove temp directory usage in plaintext-localfile

The PlaintextLocalFileStorageTest use the /tmp folder to store
new data-file and then replace original data-file in custom location.

This can cause error when move operation is performed accross
data stores. Fix this issue by storing new data-file in same
folder as original data-file.

Also removed the usage of /tmp folder in the tests.

JIRA: NETCONF-1396
Change-Id: I3124253567d9d47e59c61aa49f193980cff15d69
Signed-off-by: Peter Suna <peter.suna@pantheon.tech>

Introduce transport.http.ReadyResponse

ReadyResponse is a Response which can seamlessly translate to
HttpResponse, without throwing a checked exception.

Also promote EmptyRequestResponse to transport.http.

JIRA: NETCONF-1379
Change-Id: I8ce712a15dd692dcdd03cf00c9f92ed0be80a0d5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Modernize netconf-server-mdsal

Eliminate most of deprecation warnings by migrating to
DataObjectIdentifier.

Change-Id: I6fb2a22be4dc7c31d8f04907fa5da79fef71802a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use switch expression to assign type

Address the TODO we have left here.

Change-Id: I83191cd97b8fd2c6325325866f5f61b1483e2156
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Reduce argument passing in PathsEntity

We are passing down arguments from fields. Stop doing that and reference
the fields instead.

Change-Id: I6e3d4143707426b81a4219e11e695a742c9d497d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add OpenApiServiceImpl.unboxOrZero()

Using Objects.requireNonNullElse() operates on objects, hence we are
bouncing between Integer and int.

Add unboxOrZero(), which does what it says on the tin: unboxes non-null
and returns 0 for nulls.

Change-Id: Ic69b4284dc0ddcabf69e46d7e458970e5b771dc9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor PreparedRequest/Response class hierarchy

PreparedRequest, along with its sibling API components should really
live in transport-http.

This patch pulls it down, clreaning up the class hierchy.

JIRA: NETCONF-1379
Change-Id: Idada210d5236675037424650a52546bf6155c6a3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix SSH client re-authentication

The onKeyEstablished() callback is invoked whenever a key is
established, i.e. also after rekeying.

This leads to us attempting to trigger authentication again, which is
stopped cold by an ISE.

Fix this by bringing back the check we used to have in
NetconfCallHomeServer.

JIRA: NETCONF-1405
Change-Id: I2a15c319473f3a7a1082f5f58a9a30e9fbec8d73
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Modernize MonitoringToMdsalWriter

Switch to using DataObjectIdentifier.

Change-Id: Ib4ebe27d2bd90fd056eac627f72c153e50f35c94
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate OpenApi streams

There is no need to perform complicated buffering, as we have an
OpenApiEntityWriter.

We perform the following refactors:
- OpenApiInputStream is now DocumentEntity
- ComponentsStream is (for) now ComponentsEntity
- SchemasStream is now SchemasEntity

We also instantiate primary entities eagerly, as now they are costing us
nigh nothing and their presence makes it clearer as to what state is
encapsulated where.

JIRA: NETCONF-1370
Change-Id: I235ef442fddae47591348c2e73419972d3265c03
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix invalid use of @NonNull

Primitive types are not references, hence they can never be null.

Change-Id: Ia68c25593c3a4383970d59f5588f5ce6ba9cf2bc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate to BindingDataCodecFactory

BindingDOMCodecFactory is deprecated, use its replacement.

Change-Id: I224714448be903660b635426f6b675007d6bbf34
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Modernize yanglib-mdsal-writer

Use DataObjectIdentifier instance of InstanceIdentifier.

Change-Id: I9b4ce5f765d2a28ee0cad45cdd2e38f3a5e6ff65
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add ChunkOutputStream

Address the TODO left by the previous patch: ChunkOutputStream knows
the output and chunkSize, plus it gets a buffer.

write() methods now append to the buffer and frame it to output as
needed.

While we are here, also improve toString() implementations and add more
documentation.

Change-Id: I5cf0b91ab593bd9ab9151051bf828c9877f1386e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move EndpointInvariants

EndpointInvariants is a simple DTO, we can just move it to
impl package without exposing anything to the outside world.

JIRA: NETCONF-1379
Change-Id: I282a908b771132bcb71b5098c87f23b3f3b62fa4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move SegmentPeeler

SegmentPeeler is a utility useful to all users to transport-http, move
it there.

JIRA: NETCONF-1379
Change-Id: I281d48c548cee2752c58f13ec58320a6fddb8b8e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Finish RestconfSession/PendingRequest integration

We have all the bits and pieces concentrated in RestconfSession, now
implement request tracking as outlined in the FIXMEs, completing the
dispatch integration.

JIRA: NETCONF-1379
Change-Id: Icc5216b5496abda90cd2f1368f99134a07b09306
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Split out HTTPServerSession

RestconfSession has a top-half, which is RESTCONF-specific, and a
bottom-half, which is closely related to HTTPServer's channel setup.

This splits out HTTPServerSession, so that it can be moved to
transport-http and reused by other components if/when the need arises.

JIRA: NETCONF-1379
Change-Id: Iab96baab4b9469a6a5eaf2d3234cfdbb813a59a6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate Http2Utils

This class is a case of false sharing:

1. most of the callers of copyStreamId() are server-side, so they
   can squat in ServerSseHandler.

2. connectionHandler() is differentiated by a boolean -- which selects
   the framer and deals with the setup of listeners.

Eliminate the entire class at the cost of duplicating ~15 lines of code.
While that may sound horrible, it is actually what we want to do,
because now we can evolve the setup on client side and server side of
things separately.

JIRA: NETCONF-1379
Change-Id: I3a5f4288f4b0d65fa53cc039e37b2dde70f46e87
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate RestconfRequest

The sole remaining user of this class is RestconfSession, which uses
it just as an indirection.

Fold RestconfRequest.execute() into RestconfSession.executeRequest(), so
that we get all the things in one place.

JIRA: NETCONF-1379
Change-Id: I085d5c1e020d31ab1deb38c5496f1ace0bcd8ce7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor AbstractRequestProcessorTest wiring

This patch finishes the 'RequestDispatcher' refactoring saga by wiring
AbstractRequestProcessorTest to RestconfSession so that all requests
we send end up going through all of our validation.

This flushes out the fact we do not add a Host: header, which this patch
adds to all testing messages.

JIRA: NETCONF-1379
Change-Id: I907d689e1524b501070d2ef96a00df77ad37aa25
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Improve PendingRequest.execute()'s body handling

Empty body is not really valid in terms of RESTCONF -- in fact we have
some code to detect empty inputs in restconf-server-api.

Update PendingRequest's contract to allow for null bodies and deal with
that in PendingRequest{With,Without}Body.

With that expressiveness in place, rework RestconfSession's content
reference handling, so that we release the body as soon as the execution
is done.

JIRA: NETCONF-1379
Change-Id: Idb3b92450cabe6fd27661e70863fa8b64e3bc93a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add PendingRequestWithoutBody

We are getting our body handed as an InputStream and it is critical we
close it.

PendingRequestWithBody already deals with that, now we introduce
PendingRequestWithoutBody, which closes the stream prior to continuing
execution.

In order to prevent accidental mistakes, make AbstractPendingRequest a
sealed class allowing only these two subclasses, so that everybody else
has to pick on which side they are.

JIRA: NETCONF-1379
Change-Id: Ic06c133bb401f8ff8eeb5b1e2df55f074148a546
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate APIResource.firstSegment()

APIResource should not care how it is reached from EndpointRoot, move
that knownledge out.

JIRA: NETCONF-1379
Change-Id: I79704a1be259265048a0205db397819252dc28e1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Split out EndpointRoot

RestconfSession includes more functionality than it should, mostly
because there was no other place to stick it while we had
RestconfRequestDispatcher.

This patch introduces EndpointRoot, which acts as the semantic root
resource of a particular Endpoint, taking care of dispatching between
registered resources.

JIRA: NETCONF-1379
Change-Id: Id59841b92f5a568024463a7355d6bfdf2969a7c9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fixup OpenApiBodyWriter usage

OpenApiBodyWriter is a JAX-RS integration capable of streaming an entity
into an OutputStream, but we are misusing it to buffer entities in our
InputStreams.

Introduce OpenApiBodyBuffer to perform exactly the operation we need,
migrating complicated callers.

MetadataStream is straight-up replaced with pushing the MetadataEntity
into Response -- whence it is picked up by OpenApiBodyWriter.

JIRA: NETCONF-1370
Change-Id: Ic6f7b779b7afe28b90739351b48aff2aceedf0e0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up restconf-openapi dependencies

We are using jackson-databind only for tests, hence we can work with
just jackson-core.

Update our wiring and clean up dependencies, switch enforcement on, so
we do not regress.

JIRA: NETCONF-1370
Change-Id: Ia553b53e6d580a072eea67813062f8b27729e343
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump SSHD to 2.14.0

https://github.com/apache/mina-sshd/blob/master/docs/changes/2.14.0.md

Change-Id: I14fa8f3641550852705811a93521128768e15246
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Propagate TransportSession to NettyServerRequest

Subscriptions will need to be attaching resources to RestconfSession,
this patch ensures we expose it from ServerRequest.

JIRA: NETCONF-714
Change-Id: I7948ea3f76351de7b69ae0b99a15b20c79cfdc6f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Rename RestconfRequestDispatcher to APIResource

At this point there is only a teensy-tiny bit of the old 'dispatch()'
logic living in RestconfRequestDispatcher and it now really acts as a
the root of RESTCONF -- i.e. what RFC8040 calls 'API Resource'. Rename
it to match its function.

JIRA: NETCONF-1379
Change-Id: I18437f5346ff6c1c179a477514001a2c4bb692fa
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move RestconfRequestDispatcher processing

The entire RestconfRequestDispatcher.dispatch() thing is going to be
refactored. Move it to RestconfRequest, as it is completely independent
from RestconfRequestDispatcher.

JIRA: NETCONF-1379
Change-Id: I6a7ea7768fa467002beecc38d806898f46cca55c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up RestconfRequestDispatcher a bit

This class is becoming a normal resource, hence it should not be logging
things on behalf of the entire component.

Move the logging bits to RestconfTransportChannelListener and move the
prepare() method just below the constructor, so that it is clear what is
going away at some point.

JIRA: NETCONF-1379
Change-Id: Ia5b9f168c164ac30341ae3cdc43eb2a34b696b84
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Trim leaf resource method names

Now that each resource has its own class, we can shorten private method
names.

JIRA: NETCONF-1379
Change-Id: If2c257a3ac0fbda401ab24b12dc71539b89c03a9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

RestconfRequestDispatcher is an AbstractResource

We now have a clear definition of what a resource is -- but the
prepare() method has some tension between RestconfRequestDispatcher's
and AbstractResource's (and its subclasses') needs.

Resolve this tension by introducing AbstractLeafResource, which is what
DataResource et al. really are.

This allows us to redefine AbstractResource.prepare() to match exactly
what RestconfRequestDispatcher's idea, with AbstractLeafResource
providing the common glue during invocation.

JIRA: NETCONF-1379
Change-Id: I6e060eac0513c6188627a0543bd87708edbdbf6e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Split out RESTCONF request preparation

RestconfRequestDispatcher's organization is now such that there are two
three distinct parts to it:

1. the 'RESTCONF API root resource' part, dealing request
2. the preparation of subresources
3. the Netty pipeline part, dealing with interface to RestconfSession

In this patch we introduce AbstractResource to split out utilities and
constants shared for the first two parts. It additionally defines the
prepare() function, which we use from the last part.

We split out {Data,Modules,Operations,YLV}Resource, so that each hosts
only methods specific to them.

JIRA: NETCONF-1379
Change-Id: I0b42a3920146a30f22ce8bfdb3b5fd97693bfcdf
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor RESTCONF request dispatch

The act of servicing an HTTP request has four distinct phases:
- request validation and binding, synchronous
- request execution, asynchronous
- result interpretation, synchronous
- result streaming, asynchronous

We have first two cobbled together in RestconfRequestDispatcher, while
the last two are handled synchronously in ResponseUtils and the various
RestconfRequest subclasses.

This patch tackles the problem in RestconfRequestDispatcher, separating
the two phases.

We introduce a prepare() method, whose sole raison d'etre is to
synchronously process the contents of HttpRequest, i.e. method, URI and
headers, so as to arrive at a specific request (and its parameters) that
we want to be executing.

The result is communicated via PreparedRequest, which is a sealed
interface with two specializations:

1. CompletedRequest, holding an immediate result -- typically an error
   response, or a response that is statically known (such as the answer
   to 'OPTIONS /restconf/operations HTTP/1.1')
2. PendingRequest, represents has a number of specializations for the
   various requests we can execute asynchronously

The former is handled by immediate conversion to a FullHttpResponse,
which is then sent to the client. The latter is then fed any request
body, if present and executed, with the results processed in the usual
way.

We also eliminate two implementation hotspots:

1. RequestParameters, which has comindled the various request-related
   concerns in a structure-like blob. We now track the various
   parameters in the various AbstractPendingRequest subclasses, keeping
   them colocated with their users.

2. ResponseUtils along with the message builder -- which are now
   replaces through the Response interface, which offers decoupling of
   response shapes from how they map to HTTP constructs -- which is
   really a pipeline-side concern.

The result is that we have separated out the concerns, so that each
processing stage has a natural view of what it needs to handle --
resulting in much more modular codebase, where individual concerns can
be further evolved without touching others.

Most significantly, the process of turning RestconfServer responses
into ByteBufs is driven completely from PendingRequestListener -- i.e.
fully at the discretion of the Netty pipeline side of the house.

JIRA: NETCONF-1379
Change-Id: I58b89a389faac4a305881f425b5ec6cfac0d77f4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use exhaustive switch in sessionEvent()

We really want full coverage of possible events, use a Java 21 trick
to get that.

Change-Id: I6f7005700406ad713ea302f0ef511c74a0ba1700
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use HttpRequest in PrincipalService

PrincipalService requires access to headers, nothing else. Make sure it
does not carry the baggage of FullHttpRequest.

JIRA: NETCONF-1379
Change-Id: Icae70e153118dc661fc7a50352a6503337ac7ba2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Separate out ugly assert

We have a rather funky test here, which happened to work because we had
no guards against thrown exceptions.

Those guards are coming back, hence this late-access to InputStream will
not work.

Refactor the test harness into its own class and mark it for
replacement.

JIRA: NETCONF-1379
Change-Id: I6f233fb4dc3a223b375d0e611e12dc5d8ccd0996
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add restconf.server.ImplementedMethod

We are making a check whether a particular method is implemented via a
Set.contains(). This involves a hash-based lookup and gives us a
boolean.

We can do better: define an ImplementedMethod enum, and use Map.get()
for the same check. This has the same lookup cost, but gives us a
singleton check, which we can use for more efficient dispatch.

JIRA: NETCONF-1379
Change-Id: Ic58a0d2d5cdf836b8acd40f5be5c6f1c40ecced7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Expose AbstractServerRequest.prettyPrint()

Having this method limited to subclasses is quite limiting -- it means
that classes in the same package as a subclass cannot reference it.

Rather than requiring subclasses to provide a different method that
bridges to prettyPrint(), make the method public, which does no harm.

JIRA: NETCONF-1379
Change-Id: Ib3f307d650756e405aef2a850a2af8dd579fac1e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

OpenAPI: Actions displayed under /rests/operations

Fixed base path for action to be /rests/data instead of
/rests/operations.

JIRA: NETCONF-1395
Change-Id: Iae1043447252d3ff97b0f067a99b5b0a94fdc06d
Signed-off-by: Oleksandr Zharov <oleksandr.zharov@pantheon.tech>

Eliminate extractAcceptTypes()

This method is a simple internal intermediary, which actually hides
things from its caller. Inline it and drop a FIXME pointing out the
naivety of responseTypeFromAccept().

JIRA: NETCONF-1379
Change-Id: If22a569be1e740c64673f3675d6095126c197dba
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Expand usability of MessageEncoding

Now that MessageEncoding is a well-defined, expand its usefulness by
including:
- direct RestconfStream.Encoding constant reference
- the set of producer-compatible media types

JIRA: NETCONF-1379
Change-Id: I586773cd08449224aba5568d90cecdcede9513fc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Split out NettyEndpointConfiguration.Encoding

This encoding enumeration is backed by a well-known specification,
namely RFC8040 section 5.2 "Message Encoding".

Promote this class to a top-level construct, adding the appropriate
documentation to ensure where this is coming from.

Update propagation within o.o.restconf.server, so that we do not
lose type safety to AsciiString.

Furthermore split out the from(String) method to restconf-nb, as its
contract is undocumented and is really only appropriate in parsing
the configuration file there.

JIRA: NETCONF-1379
Change-Id: I60499d255d94463e498d6aa1511715469d1b8ead
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Split out bootstrap factory

NettyEndpoint currently allocates its own BootstrapFactory and it fails
to clean it up.

Furthermore the configuration prevents users from sharing threads
between multiple subsystems as well as using split groups fro accepting
connections (boss) and servicing connections (worker).

Fix the leak and plug the capability gap by more closely following the
netconf-nb model -- and have the BootstrapFactory managed separately
outside of NettyEndpointConfiguration.

JIRA: NETCONF-1379
Change-Id: I2cd0e93fc29f1413b3b4438ce74a3327f72f3f1a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Retain Encoding in endpoint configuration

Do not peel Encoding, just forward requests for default encoding to it.

JIRA: NETCONF-1379
Change-Id: I8df2f190b030f505023ac6a5e28aeca54eb74343
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add response body example for ROOT operations GET

Hardcoded top-level container "ietf-restconf:operations" for
the OpenAPI-generated example.
Implemented fixed structure to ensure proper schema generation
for both JSON and XML.
Updated unit tests (UT) accordingly to reflect
the new changes.

JIRA: NETCONF-1343
Change-Id: I4aa9f4d782eee929cd49b445913683c3cd1ea33b
Signed-off-by: Yaroslav Lastivka <yaroslav.lastivka@pantheon.tech>