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

Public Release for Fall24 meta-release #101

Merged
merged 11 commits into from
Sep 5, 2024
Merged

Public Release for Fall24 meta-release #101

merged 11 commits into from
Sep 5, 2024

Conversation

bigludo7
Copy link
Collaborator

@bigludo7 bigludo7 commented Sep 4, 2024

According to the discussion in: V0.2.0 M4 by FabrizioMoggio · Pull Request #95 · camaraproject/CallForwardingSignal (github.com)

M4 version for Fall24:

What type of PR is this?

Add one of the following kinds:

  • subproject management
  • cleanup
  • documentation

What this PR does / why we need it:

final release for Fall24

Which issue(s) this PR fixes:

#94
#93
#96
#97
#98
#100

Special notes for reviewers:

PR done on behalf of @FabrizioMoggio

Changelog input

 release-note

Additional documentation

This section can be blank.

docs

Readme copied from @FabrizioMoggio PR + fixed @tanjadegroot issues
Copied from @FabrizioMoggio PR + fixed api name in link as proposed by @tanjadegroot
Fixed api name in links
Copied from @FabrizioMoggio PR + fixed api yaml name + used link
Copied from @FabrizioMoggio + fixed type in line 229 (@tanjadegroot )
Copy link

github-actions bot commented Sep 4, 2024

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 2 0 0.02s
✅ OPENAPI spectral 1 0 1.67s
✅ REPOSITORY git_diff yes no 0.01s
✅ REPOSITORY secretlint yes no 0.66s
✅ YAML yamllint 1 0 0.33s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@bigludo7
Copy link
Collaborator Author

bigludo7 commented Sep 4, 2024

@tanjadegroot For context: @FabrizioMoggio has issue with easyCLA so I copied his PR.
I fixed your review comments.
Regarding your 2 questions from the yaml:

on line 229: does unconditional forwarding overwrite conditional forwarding if both are active ?
Ludo: I tend to think that unconditional has predominance and override all other but happy to listen from @FabrizioMoggio

on line 230: with "This endpoint exceeds the main scope of the CFS API, for this reason an error code 501 can be returned" do you mean that this second endpoint is optional and may not be provided and that 501 is returned in that case ?
Ludo: Yes

Copy link
Contributor

Choose a reason for hiding this comment

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

@bigludo7 the filename need to be changed to call-forwarding-signal.yaml (as done in #95)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes ! done !

CHANGELOG.md Outdated
Comment on lines 94 to 96
- OpenAPI [YAML spec file](https://github.com/camaraproject/CallForwardingSignal/blob/r1.1/code/API_definitions/call-forwarding-signal.yaml)
- [View it on ReDoc](https://redocly.github.io/redoc/?url=https://raw.githubusercontent.com/camaraproject/CallForwardingSignal/r1.1/code/API_definitions/call-forwarding-signal.yaml&nocors)
- [View it on Swagger Editor](https://editor.swagger.io/?url=https://raw.githubusercontent.com/camaraproject/CallForwardingSignal/r1.1/code/API_definitions/call-forwarding-signal.yaml)
Copy link
Contributor

Choose a reason for hiding this comment

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

@bigludo7 here in r1.1 the filename will stay unchanged ... the links work and will work perfectly. Don't try to change the history 😄

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed .... but we do not have an issue fir r1.2-rc because not sure where the links points to.

CHANGELOG.md Outdated Show resolved Hide resolved
bigludo7 and others added 3 commits September 5, 2024 08:14
Co-authored-by: Herbert Damker <herbert.damker@telekom.de>
updated the link for the r1.1
@FabrizioMoggio
Copy link
Collaborator

@tanjadegroot For context: @FabrizioMoggio has issue with easyCLA so I copied his PR. I fixed your review comments. Regarding your 2 questions from the yaml:

on line 229: does unconditional forwarding overwrite conditional forwarding if both are active ? Ludo: I tend to think that unconditional has predominance and override all other but happy to listen from @FabrizioMoggio

on line 230: with "This endpoint exceeds the main scope of the CFS API, for this reason an error code 501 can be returned" do you mean that this second endpoint is optional and may not be provided and that 501 is returned in that case ? Ludo: Yes

_on line 229: my sentence was actually intended to just say that more than one kind of call forwarding service can be active at the same time and that the API will provide back every kind of service that is configured. The main kind are unconditional and conditional than we can have different flavors of conditional. The logic of what can be configured in the Network and the priorities of the call fwd services have no effect on the API. Anyway I agree, if the they are both active in the same timeslot, unconditional takes the precedence.

-on line 230: some additional information here: #37 (comment)

Copy link
Collaborator

@FabrizioMoggio FabrizioMoggio left a comment

Choose a reason for hiding this comment

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

README.MD to be updated (we should remove or line 21 or line 23)

@@ -15,23 +15,22 @@ Repository to describe, develop, document and test the Call Forwarding Signal AP
* The Application Server invokes the Call Forwarding Signal API to determine if a specific phone number has an active "call forwarding" setup. "Call forwarding" is a network service that redirects incoming calls to another phone number (configured in the service). The Call Forwarding Signal API can be used by a bank to verify if a “call forwarding” option is active on the customer’s phone number to avoid frauds. A call from the bank to the customers can indeed be forwarded to a different number because of a fraud attempt.
* The API is useful to avoid frauds. Banks offer the option for users to receive telephone calls to solve problems, e.g. in case of issues with the bank App, issues with Smartphones, the bank website or any tool providing access to the online bank account management system.<br>How does the fraudster operate?<br>• Through social engineering, the fraudster gathers information about bank customers who are engaged in such a situation, needing the bank to contact them.<br>• Once they identify the service provider and phone number of a customer, they manage to successfully activate, on the customer profile, the 'call forwarding' to a phone number they control.<br>• After successfully setting up 'call forwarding,' the fraudster then contacts the bank, claiming an inability to access online banking services and requesting a call from a bank representative. <br>• When the bank calls the registered contact number of the user, the active 'call forwarding' diverts the call directly to the fraudster, thus completing their scheme.<br> The Call Forwarding Signal API can be used by the bank to verify if a "call forwarding" option is active on the customer's phone, avoiding such a fraud.
* Describe, develop, document and test the APIs (with 1-2 Telcos)
* Started: February 2024
* Location: virtually
* Started: February 2024

## Meetings
* Meetings are held virtually
Copy link
Collaborator

Choose a reason for hiding this comment

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

@bigludo7 we should remove or line 21 or line 23

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

Remove line 23
Copy link
Collaborator

@FabrizioMoggio FabrizioMoggio left a comment

Choose a reason for hiding this comment

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

LGTM

@FabrizioMoggio FabrizioMoggio mentioned this pull request Sep 5, 2024
@FabrizioMoggio FabrizioMoggio merged commit 183e1c0 into main Sep 5, 2024
2 checks passed
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.

3 participants