Skip to content

Commit

Permalink
http-api: T6736: add migration script and update version
Browse files Browse the repository at this point in the history
  • Loading branch information
jestabro committed Sep 30, 2024
1 parent 6dbacab commit 5a5d24f
Show file tree
Hide file tree
Showing 2 changed files with 44 additions and 1 deletion.
2 changes: 1 addition & 1 deletion interface-definitions/include/version/https-version.xml.i
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<!-- include start from include/version/https-version.xml.i -->
<syntaxVersion component='https' version='6'></syntaxVersion>
<syntaxVersion component='https' version='7'></syntaxVersion>
<!-- include end -->
43 changes: 43 additions & 0 deletions src/migration-scripts/https/6-to-7
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
# Copyright 2024 VyOS maintainers and contributors <[email protected]>
#
# This library is free software; you can redistribute it and/or
# modify it under the terms of the GNU Lesser General Public
# License as published by the Free Software Foundation; either
# version 2.1 of the License, or (at your option) any later version.
#
# This library is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
# Lesser General Public License for more details.
#
# You should have received a copy of the GNU Lesser General Public License
# along with this library. If not, see <http://www.gnu.org/licenses/>.

# T6736: move REST API to distinct node


from vyos.configtree import ConfigTree


base = ['service', 'https', 'api']

def migrate(config: ConfigTree) -> None:
if not config.exists(base):
# Nothing to do
return

# Move REST API configuration to new node
# REST API was previously enabled if base path exists
config.set(['service', 'https', 'api', 'rest'])
for entry in ('debug', 'strict'):
if config.exists(base + [entry]):
config.set(base + ['rest', entry])
config.delete(base + [entry])

# Move CORS settings under GraphQL
# CORS is not implemented for REST API
if config.exists(base + ['cors']):
old_base = base + ['cors']
new_base = base + ['graphql', 'cors']
config.copy(old_base, new_base)
config.delete(old_base)

0 comments on commit 5a5d24f

Please sign in to comment.