Skip to content
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.

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

Bump dependencies #366

Merged
merged 4 commits into from
Feb 2, 2024
Merged

Conversation

lo-simon
Copy link
Contributor

Bump boost to 1.83.0 and cpprestsdk to 2.10.19

// highest_pri, lowest_pri [registry, node]: used to specify the (inclusive) range of suitable 'pri' values of discovered APIs, to avoid development and live systems colliding
// highest_pri, lowest_pri [node]: used to specify the (inclusive) range of suitable 'pri' values of discovered APIs, to avoid development and live systems colliding
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this an existing error in the comment that's been corrected?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, the comment in it is corrected.

When IS-10 was first instructed, the higher_pri and lowest_pri were also used for Authorization API priority, but while under development, very often the Registration API can be using a different pri than the Authorization API, as the Authorization API does not provide by the Registry. So therefore in settings, there are a new authorization_highest_pri and authorization_lowest_pri for the Authorization API priority.

Copy link
Contributor

@jonathan-r-thorpe jonathan-r-thorpe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jonathan-r-thorpe jonathan-r-thorpe merged commit 5866061 into sony:master Feb 2, 2024
11 checks passed
@jonathan-r-thorpe jonathan-r-thorpe deleted the bump-dependencies branch February 2, 2024 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants