Specialize HTTPClient

We know statically whether a particular client is TLS-enabled, so use
two subclasses, each specialized for the task.

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

Move ResponseUtils.optionsResponse()

This method is only called from RestconfRequestDispatcher. Move it there
and simplify it by bypassing ResponseBuilder's complexities.

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

Clean up NettyServerRequest subclasses

Almost all anonymous NettyServerRequest subclasses are inefficient in
that they retain RequestParameters twice:
- once in NettyServerRequest
- once via reference to 'params' method argument

Make sure we reference NettyServerRequest.requestParams instead of
params, reducing memory overhead of each request.

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

Add FormattableServerRequest

We have two distinct NettyServerRequest subclasses doing exactly the
same thing. Share a single FormattableServerRequest class between the
two code paths.

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

Introduce RestconfRequest

The use of FutureCallback is misleading and not very well controlled.
Introduce RestconfRequest, which shows that the onFailure() path is
never taken.

This also provides us with an anchor point for future evolution, as we
must be mindful of asynchronous processing. The requirements are laid
down in a FIXME, which we will address in future improvements.

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

Do not pass around RestconfServer

Now that we have localized request dispatch, we can access
RestconfServer via RestconfRequestDispatcher's field rather than passing
it around to static methods.

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

Limit the number of candidates for string example

The number of candidates tried when a string example is generated
with regex & length constraints may increase exponentially as the
string length increases and this may cause out-of-memory error.

For example, if there are 10 transition paths at each step, then
the number of candidates becomes 1 million after just 6 characters.

Limit the maximum number of transitions allowed at each step to prevent
the exponential growth in search paths. This reduces the chance of
generating a conformant example a little, but prevents excessive use
of memory and CPU cycles.

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

Do not use LinkedList in plaintext-*

Use an ArrayList instead.

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

Eliminate RequestUtils

The only production caller is RestconfRequestDispatcher, move these
utility methods there.

Hide them and ensure AbstractRequestProcessorTest uses explicit
calls to ApiPath.parse().

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

Eliminate ModulesRequestProcessor

RestconfRequestDispatcher can host the static methods just as well, and
such organization is actually conducive to structuring dispatch based on
needs, rather than upfront parsing.

Also updates RFC references to point to their canonical web presence.

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

Eliminate OperationsRequestProcessor

RestconfRequestDispatcher can host the static methods just as well, and
such organization is actually conducive to structuring dispatch based on
needs, rather than upfront parsing.

Also updates RFC references to point to their canonical web presence.

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

Eliminate DataRequestProcessor

RestconfRequestDispatcher can host the static methods just as well, and
such organization is actually conducive to structuring dispatch based on
needs, rather than upfront parsing.

Also updates RFC references to point to their canonical web presence.

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

Clarify FIXME for NETCONF-714

Clarify FIXME in NettyServerRequest to be part of NETCONF-714.

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

Fix testtool request failure to netty endpoint

Testtool uses java's HttpClient which includes
HTTP/2 upgrade headers by default when PATCH requests
being sent. This results 413 Entry too large response
from Netty-based server because HttpServerUpgradeHandler
is not expecting non-empty request body with
protocol upgrade request.

The fix limits protocol version for HttpClient to
HTTP 1.1 to omit upgrade headers.

JIRA: NETCONF-1376
Change-Id: I2c07edce78b521417cf073f5c0d757e58f557c91
Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>

Fix 401 Response headers for HTTP Transport

Auth handler updated to return WWW-Authenticate
header with 401 Unauthorized response.

JIRA: NETCONF-1394
Change-Id: I018b7c37ec08eae4525355001265f4151bafa298
Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>

Bump versions to 8.0.3-SNAPSHOT

This starts the next development iteration.

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

Release netconf

Dispatch OPTIONS requests to RestconfServer

RestconfServer is providing guidance as to what kind of resource a
particular path represents. We take advantage of this, to ensure parity
with JaxRsRestconf.

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

Fixup OptionsResult

We need to differentiate between RPCs and actions, an unfortunate
side-effect of difference of their invocation modeling.

Split OPERATION into ACTION and RPC, and report them as appropriate.

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

Adapt docs for netconf-node state encapsulation

Adapt user guide to changes caused by netconf-node state
encapsulation.

JIRA: NETCONF-1388
Change-Id: I477f14e814c0d4dfa3833454b8803c2a72fdc18e
Signed-off-by: Samuel Schneider <samuel.schneider@pantheon.tech>

Fix stream URLs for Netty RESTCONF

Restconf URI passed for URL construction require to be ended
with slash otherwise last path element will be dropped by URI#resolve.

JIRA: NETCONF-1390
Change-Id: I1707b8460741bd99a971359fd152152b60440c8c
Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Translate RpcCredentials with a switch expression

We have an if/else cascade here, use a switch expression instead.

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

Encapsulate netconf-node state

Encapsulate netconf device related configuration into its own container.
So mandatory credentials requirement is not enforced for anyone who is
not writing that container.

This allows using other custom topologies without issues.

JIRA: NETCONF-1382
Change-Id: If015222df4b4a885c832f8de35f3c482e9dfbc2a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Samuel Schneider <samuel.schneider@pantheon.tech>

Add {data,operations}OPTIONS() methods

JAX-RS endpoint is doing a number of bad things w.r.t OPTIONS:
- it does not produce Accept-Patch header
- it exposes implementation structure via WADL
- it is not accurate for /data/* resources

Enrich RestconfServer API to allow querying the nature of a particular
data item and existence of operations -- which in turn allows us to
report accurate OPTIONS.

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

Set correct SchemaInferenceStack for list example

In the method prepareFirstListExample, SchemaInferenceStack points
to the list, which is incorrect as we are processing its child.

This method is called only when the min-elements constraint is set.

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

Fix string example generation error

Following a single regex transition path does not guarantee an example
conforming to the regex & length constraints.

Search all transition paths adding a character at a time until the
minimum length requirement & regex pattern are satisfied or the maximum
length limit is reached.

This guarantees the shortest possible example that satisfies the regex
& length constraints.

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

Eliminate Http2SettingsFilter

We need Http2SettingsFilter only because Http2Utils.connectionHandler()
configures Http2ToHttpAdapter to send it out.

Do not override the default configuration, leading to HTTPClient's
adapter not sending these -- and hence we can ditch the filter.

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

Clean up restconf-nb dependencies

The dependencies are quite inaccurate now, clean them a up a bit.

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

Clean up odl-restconf-nb

Clean up packaging, so we do not pull in duplicate bundles.

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

Add odl-restconf-(server-)api

Package restconf-api as a separate feature, so that it can be used from
other places.

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

Move restconf-server-jaxrs tests

Move tests into the artifact they are actually testing.

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

Package transport-http in odl-netconf-transport

transport-http does not have a home. Make sure to include it in
odl-netconf-transport.

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

Integrate transformation to NettyServerRequest

Make NettyServerRequest abstract, with a transform() method. This makes
it clear what specializations we have and also reduces the amount of
objects we allocate.

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

Slice XRD/JRD resource

We need to use ByteBuf.slice() so the reader index on shared buffer does
not get updated.

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

Use a switch expression with DataPostResult

We have a sealed hierarchy, hence using a switch expression is
exhaustive and we can eliminate an impossible case. The same thing
applies to both RESTCONF endpoints.

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

Use OPTIONS * for upgrade

GET / is a resource operation, which may result a ton of data, be
subject to access control, or actually be unimplemented.

Use OPTIONS *, which is well-documented to be a 'no-op' kind of HTTP
request.

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

Advertize "defaults" RESTCONF capability

Advertizement of "defaults" RESTCONF capability is mandatory, yet we do
not do that.

Advertize it with basic-mode=explicit, which most closely matches the
semantics of the MD-SAL datastore.

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

Do not use karaf-services-maven-plugin

We are not really using services -- just publishing Karaf commands.

Add the MANIFEST header and ditch use of karaf-services-maven-plugin --
fixing a build-time warnings and eliminating the unnecessary code it
packages in the bundle.

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

Fix truststore-api dependencies

We are using nullness annotations, declare them to fix a warning.

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

Fix dependency issues in keystore/*

We have two dependency issues reported here, fix them up.

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

Fix create-notification-stream implementation

We have a mismatch in the output leaf name, as sal-remote.yang has a
day-0 modeling bug: what was supposed to be 'stream-name' is named
'notification-stream-identifier' instead.

Fix the implementation to use the correct container.

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

Eliminate ResponseUtils.allowHeaderValue()

This creates a static string, from a set of methods. Rather than
performing magic, let's use hand-crafted equivalents.

Also do not share these strings with tests, but rather have tests assert
them separately.

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

Eliminate restconf.server.Method

This enum might look like it is useful until, but it really is not: we
can equally well use HttpMethod and switch on HttpMethod.name().

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

Split out WellKnownResources

Well-known resources are extremely simple to service, as they take only
a few inputs and produce an immediate response.

Split them out into their own class and execute them immediately from
RestconfSession. This allows us to report '501 Not Implemented' for
unimplemented methods, along with 'Allow' header hinting at what is
implemented.

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

Take ownership of request message

We have asynchronous processing here, hence
SimpleChannelInboundHandler's defaults are working against us.

Switch off autoRelease, hence we do not need to issue retain().

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

Expose ServerSseHandler

Rather than hiding this handler behind a simple SseUtils method, expose
it to the outside world, so it can be used directly by NettyRestconf's
initialization code.

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

Disconnect HTTPServer from RequestDispatcher

RequestDispatcher is useless layering violation sitting between Netty
pipeline and RESTCONF protocol implementation.

Do not use RequestDispatcher in HTTPServer and instead use
onTransportChannelEstablished() to wire a RestconfSession into the
pipeline.

This means we have centralized protocol-level initialization at the
correct layer and a proper class interfacing between a particular Netty
channel and the RestconfServer.

RestconfRequestDispatcher now becomes a private implementation detail,
acting as a the common router between RestconfSession and
RestconfServer (et al.).

A follow-up patch will deal with exposing RestconfSession from
NettyServerRequest.

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

Rename CreateNotificationStreamRpcTest

This is really testing CreateDataChangeEventSubscriptionRpc, rename it
accordingly.

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

Clean up CreateNotificationStreamRpc

Use CreateNotificationStreamOutput instead of
CreateDataChangeEventSubscriptionOutput, just for consistency's sake.

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

Fix ietf-restconf.yang reference

We have a typo in Javadoc, fix that.

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

Advertize :yang-patch capability

We support yang-patch media type, hence we should be advertizing that
capability.

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

Deprecate capabilityUri() methods

We have the capability strings centralized in CapabilityURI class, hence
we do not need them in individual parameters. Deprecate them for
removal.

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

Add restconf.api.CapabilityURN

We are missing a few key capabilities, simply because we are exposing
only capabilities tied to query parameters.

This patch adds restconf.api.CapabilityURN with current contents of the
corresponding IANA registry, along with ODL-specific capabilities --
mirroring what we have in netconf.api.CapabilityURN.

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

Factor out RestconfTransportChannelListener

Pipeline initialization for transport channels sits in an anonymous
class, make it non-obvious as to what its actual role should be.

Split it out into RestconfTransportChannelListener, making it
independent of NettyEndpoint.

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

Fixup restconf-server-mdsal module definition

We require the ietf-restconf-monitoring module, make sure we advertize
that.

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

Fix host meta template constants

The strings should be proper constants, so they can be inlined.

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

Refactor HTTPServer pipeline initialization

The pipeline should be initialized immediately when the channel is
brought up, so that we do not require additional client-side operations
before it is visible the the server side.

This necessitates smarter operations on the pipeline, so that we keep
RequestDispatcher at the last position.

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

Fix heartbeat overflow

sseHeartbeatIntervalMillis is an Uint32, hence it is wrong to assume its
value can safely fit in an int.

Use .toJava() instead, which returns a proper long, exposing the fact
that ServerSseHandler is using the wrong type.

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

Clean up OSGINorthbound.newNettyEndpointProps()

We have a magic check for URI scheme -- i.e. if we grow a different
transport, we might happily be using 'http' without ever realizing it.

Refactor logic to make it absolutely clear which cases we handle, so
that there can be no mistakes.

Also clean up obvious gaps in semantics of objects we create and add
FIXMEs/TODOs for follow-ups.

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

Use simple concatenation in SseUtils.chunkOf()

Using a simple concatenation improves performance. It also eliminates
the need for @SuppressFBWarnings and making transport-http independent
of spotbugs-annotations.

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

Enforce strict stubbing in transport-tcp

Make sure our mocks are throughly stubbed by pulling in
mockito-configuration.

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

Separate out Http2SettingsFilter

Http2Utils.clientSettingsHandler() is a stateless thing. Separate it out
into a dedicated class, which has a singleton instance shared across all
channels that need it.

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

Split up HTTPServer

HTTPServer is either backed by TCP or TLS, each of which has a different
flow setup. Specialize the two so we do not need to muck around with
SslHandler.

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

Clean up HTTPTransportStack, part two

Eliminate HttpChannelInitializer, as both implementations can easily be
integrated into HTTP{Client,Server}.

This also eliminates needless juggling of channel pipeline, as the
channel is always functional by the time we are initializing.

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

Clean up HTTPTransportStack, part one

A number of cleanups:
- channelInitializer should be private
- naming should be consistent
- superfluous annotations should not be present

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

Expose {SSH,TLS}TransportChannel

Knowing the identity of underlying TransportChannel is needed to
accurately report their type. Expose the two implementations, as
{TCP,HTTP}TransportChannel are already exposed.

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

Clean up HTTPTransportChannel

This class should be final and should only be instantiated from the same
package.

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

Use enhanced switch for transport dispatch

We have two supported possibilities, use a switch to discern them.

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

OpenAPI: add metadata endpoint

Added endpoints for pagination metadata.

/single/meta(?offset=0&limit=5)
/mounts/{instance}/meta(?offset=0&limit=5)

Created MetadataStream and MetadataEntity to provide metadata
which would be valuable for end users.

JIRA: NETCONF-1344
Change-Id: I14e476ac4266e48953e9faaca80782fa891d03d3
Signed-off-by: lubos-cicut <lubos.cicut@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Bump versions to 8.0.2-SNAPSHOT

This starts the next development iteration.

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

Release netconf

Bump upstreams

Adopt:
- odlparent-14.0.3
- infrautils-7.0.3
- yangtools-14.0.4
- mdsal-14.0.2
- controller-10.0.2
- aaa-0.20.1

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

Fix superfluous nullness annoutations

We have @NonNullByDefault -- there's no point in @NonNulls.

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

Add missing return

We should not be invoking callbacks twice -- make sure to return.

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

Advertize :interleave capability

We support interleave, i.e. concurrent use of RPCs and notifications.
Advertize this fact.

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

Revert "Check for netconf interleave capability"

This reverts commit c2cfd0608b41775f9e71904de14b453ad9d2ca59, as it does
not present a complete solution.

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

Do not copy constant bytebufs

Three things:
- use plain byte[] to make it clear we are using US_ASCII
- ensure the constant is a read-only buffer
- use .retainedSlice() instead of .copy()

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

Convert restconf-server to bnd-parent

This is a simple artifact, make sure we use bnd-parent substrate to keep
it modern.

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

Fix Date import slopiness

We are using java.util.Date, not java.sql.Date.

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

Clean up restconf-server dependencies

We have a ton of unused dependencies here, clean them up.

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

Revert "Fix jira link"

This reverts commit c5f2790eed5c7989d74191e0b2b92474c897849c, as the
underlying problem has been resolved.

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

Split out JavaCommonCounters

We are maintaining a native view of CommonCounters, which are defined to
be wrapping unsigned 32bit counters.

Our intermal use does not correctly truncate these, as the original code
got confused by 'Long' representing 'uint32' in Binding long time ago.

There is no need to store these counters in long form -- we can convert
that on demand via a newly-introduced JavaCommonCounters -- which extend
CommonCounters, i.e. the Binding interface generated for the grouping.

Add two implementations, mutable and immutable, and expose the ability
to increment as MutableJavaCommonCounters.

This allows us to to expose the read-only view from NetconfServerSession
without giving users access to directly modify them -- making the class
more convenient.

One side effect of us implementing CommonCounters is that we can use
SessionBuilder(CommonCounters) constructor, which performs an automatic
initialization to current state.

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

Derive SourceHost from Channel.remoteAddress()

This removes reliance on hello backchannel for source host information.
It ends up removing a chunk of code.

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

Deprecate NetconfHelloMessageAdditionalHeader

This class should not exist, but getting rid of it requires us to do
quite some work, which in turn has impacts in netconf-{codec,netty-util}
and perhaps somewhere else as well.

Deprecate this class for removal to drive its replacements. Also provide
a lot of FIXMEs explaining the various changes to
netconf-{api,codec,client,server} that need to happen to solidify the
pipeline layout, the fool-proof the end user APIs and provide reliable
operation.

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

Refactor ThreadLocalTransformers

Rather than being a plain holder class with duplicated code for each
constant, rename the class ThreadLocalTransformer so that it extends
ThreadLocal<Transformer>, being abstract and requiring instances to
provide configure() method.

The instances themselves have a nice place in MessageWriter, as that
is where they are used.

Since we have a dedicated class and constants, these replace the boolean
-- instead we store the reference to the ThreadLocalTransformer instance
which produces corresponding transformers.

The equivalent of getters is provided by ThreadLocalTransformer.get(),
i.e. getPrettyTransformer() becomes PRETTY_TRANSFORMER.get() without any
intermediates.

Since we are in the area, let's not access the thread local immediately
-- expose threadLocalTransformer(), which just does ThreadLocal.get(),
so that subclasses defer access to when they actually need it.

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

Move {Hello,XML}XMLMessageWriter

XMLMessageWriter is a well-defined writer, independent of anything else
in netconf.nettyutil. Move it to netconf.codec, whence it can be freely
used without bringing in nettyutil baggage.

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

Refactor delayed MessageWriter replacement

AbstractNetconfSession has a Runnable extension to allow for replacing
the MessageWriter after the next message is processed.

This is really related to DefaultStartExi, which is driving this change
-- but the change can only happen *after* the RPC response is encoded
using current writer.

Make this transition more explicit by moving the mechanics to
MessageEncoder.

This improves message send, as we do not need to hook to every write
just to update it. It also makes it more accurate: the writer needs to
be switched just after the message is *encoded*, not when is written to
wire.

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

Refactor FrameEncoder

We have MessageEncoder and FrameEncoder working in tandem in the
pipeline. While this is fine and dandy from design perspective, it
leaves a bit left to be desired.

The problem is that we end up buffering the raw content of the message
into one ByteBuf (in MessageEncoder) and then copy it to another ByteBuf
(in FrameEncoder).

This is wasteful for FramingMechanism.EOM, which just wants to write the
message out and then append a fre more bytes.

Refactor FrameEncoder into FramingSupport class, which exposes a
writeMessage(), which takes a MessageWriter and a NetconfMessage and
encodes/frames it according to its rules as an atomic operation.

This removes double buffering from EOM framing, but keeps things as they
are for CHUNK framing -- with a note to improve it later.

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

Split MessageEncoder up

MessageEncoder has currently two functions:
- it receives a NetconfMessage from the pipeline
- it encodes it into a ByteBuf
- it passes the ByteBuf down the pipeline

Refactor MessageEncoder.encodeTo() to a separate class, MessageWriter,
which handles the details of how to write a NetconfMessage into an
OutputStream.

This makes MessageEncoder a final class, which has the notion of which
MessageWriter it is currently using -- and that can be switched from the
pipeline context.

This completely eliminates the need for modifying the pipeline when
switch XML/EXI and provides for nice integration surface for further
consolidation.

This entire exercise means we have only four objects dealing with
default XML encoding -- be it pretty-printing or otherwise.

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

Improve OpenAPI parameters validation

Check nullability of width, depth, offset and limit at beginning
and pass them further like int instead of Integer.

JIRA: NETCONF-1346
Change-Id: I22388eff543d25b404736d45c2ca554a61e0b497
Signed-off-by: Matej Sramcik <matej.sramcik@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Fix YangErrorsBody propagation

YangErrorsBody needs to be wrapped with JaxRsFormattableBody to
correctly match the writer, which needs pretty-print information.

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

Fix jira link

jira.opendaylight.org indirection does not work correctly, hence our tox
verify is broken. Fix it by explicitly linking to
lf-opendaylight.atlassian.net.

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

Acquire JAX-RS Principal eagerly

SecurityContext's methods only work while we are executing in the JAX-RS
framework thread. This means we need to extract the principal (and any
permission-related info) before the request execution enters its
asynchronous phase.

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

Wrap YangPatchStatusBody with JaxRsFormattableBody

JAX-RS handling of FormattableBody requires use of JaxRsFormattableBody,
as we need the pretty-print parameter. Previous conversion has missed
the case of YANG patch.

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

Adapt user guide to Netty changes

Update netconf user guide to Netty based endpoint settings.

JIRA: NETCONF-1363
Change-Id: Id7c2712d37e3a7dad919e4acbfa1ae6876e30a43
Signed-off-by: lubos-cicut <lubos.cicut@pantheon.tech>

Netty based Restconf server

- Init via restconf-nb OSGi service, TLS supported
- AAA Shiro integration for Basic Auth and Principal delivery
- SSE overlay adapter integrating RestconfStream.Registry
- Request dispatcher processing for restconf resource URIs
  plus /modules and resource discovery URIs
- error responses using restconf model

JIRA: NETCONF-873
Change-Id: I18b73302973631914f402997f0bf588b3bea7676
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Signed-off-by: Matej Sramcik <matej.sramcik@pantheon.tech>
Signed-off-by: lubos-cicut <lubos.cicut@pantheon.tech>
Signed-off-by: Oleksandr Zharov <oleksandr.zharov@pantheon.tech>
Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>

Use empty line as message separator in SSE stream

The double end-line as a message separator wasn't clearly described
in a spec, so it was missed initially (NETCONF-1339).

The external clients like postman rely on this separator
and it makes it consistent with jaxrs implementation.

Also:
- fixed javadoc reference
- lowered log level for every connection message
- using proper ping message wrapper

JIRA: NETCONF-1366
Change-Id: I0cf9cac5cf707315928e0e739f3ad5776271fbc6
Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Fix top level element insert failure

Non-null empty path identifier points to abstract root,
using it as data node identifier results validation error.

JIRA: NETCONF-1359
Change-Id: I2a66700ac5578990ed956de10f23155cf4f3c675
Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>

Use simple WebContext.name()

OSGi HTTP Whiteboard requires symbolic-name for names, otherwise we can
wreck a ton of things. Simplify the names.

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

Simplify NetconfOperationChainedExecution

There is only a single implementation that reports being a termination
point. Rather than having these checks, just use 'null' to indicate
there is no subsequent operation.

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