Skip to content
Snippets Groups Projects

Stfaseid http refactor

Merged Ghost User requested to merge stfaseid_http_refactor into develop

Description

Draft for new API. Needs extensive manual testing. Only merge if changes in https://code.fbi.h-da.de/danet/api/-/tree/10-refactor-current-proto-definitions-stfahelm and upcoming changes to CLI are approved.

Related Issue

#141 (closed)

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
Edited by Ghost User

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ghost User
  • Author Contributor

    Linter doesn't seem to be happy yet: https://code.fbi.h-da.de/danet/gosdn/-/jobs/368948

  • Ghost User added 1 commit

    added 1 commit

    • 3f85d0ee - Remove TODO comment as task is done

    Compare with previous version

  • Ghost User
  • Ghost User added 1 commit

    added 1 commit

    • 8c0bfd0a - String replacement for path in GetPath()

    Compare with previous version

  • Ghost User added 2 commits

    added 2 commits

    • 16f1f410 - changed var name to fit naming convention
    • b0ad2a76 - Merge branch 'stfaseid_http_refactor' of code.fbi.h-da.de:danet/gosdn into stfaseid_http_refactor

    Compare with previous version

  • Ghost User added 2 commits

    added 2 commits

    • b9580113 - Linter pleasing
    • 806ec67f - Merge branch 'stfaseid_http_refactor' of code.fbi.h-da.de:/danet/gosdn into stfaseid_http_refactor

    Compare with previous version

  • mentioned in issue #167 (closed)

  • Ghost User mentioned in issue #168

    mentioned in issue #168

  • Ghost User resolved all threads

    resolved all threads

  • Ghost User added 1 commit

    added 1 commit

    • fe27e00a - Fix: HTTP API test did not wait for server to start properly

    Compare with previous version

  • Ghost User added 1 commit

    added 1 commit

    Compare with previous version

  • Ghost User added 1 commit

    added 1 commit

    • 47052145 - fix: integration tests should no longer fail

    Compare with previous version

  • Ghost User added 32 commits

    added 32 commits

    • 47052145...44b946b1 - 11 commits from branch develop
    • 0aabcaf4 - updated types, states, etc. according to naming conventions, WIP
    • 39fe4769 - updated service and server names according to name changes of refactored api
    • 3afcf4f9 - updated pnd api to fit refactoring, maybe change to list version, WIP
    • b47ff23c - updated api calls for server/pnd
    • f2e745bb - updated api calls for server/core
    • 82d5eb42 - updated api/grpc according to other api changes
    • b28ea3e3 - fix for error with changeresponse and wron initial southboundtype
    • f5d2c229 - generated new mock file to fit api changes
    • ba5a06ee - fix for error with southbound types and error handling, api, pnd and core test are WIP
    • 7e8c1c4f - fix test case wrong if clause
    • 800e4c5a - tests and and minor changes for request logic correctly, WIP
    • be26ce24 - added missing method SetSbiList
    • 7923de92 - fixed get Pnd test to fit new api
    • 166f9769 - Remove TODO comment as task is done
    • 15223e20 - String replacement for path in GetPath()
    • 9be63670 - Linter pleasing
    • dcc6af04 - changed var name to fit naming convention
    • a31d9051 - Fix: HTTP API test did not wait for server to start properly
    • d67ec889 - linter pleasing
    • 32839ba3 - fix: integration tests should no longer fail
    • f54e3b0b - Merge branch 'stfaseid_http_refactor' of https://code.fbi.h-da.de/danet/gosdn...

    Compare with previous version

  • Ghost User added 1 commit

    added 1 commit

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading