<p>Hello,<br>
thank you for your contribution! I did a quick look to the code and noticed a few simple areas that should refined before going looking deeper in the functionality.</p>
<ul>
<li>Please remove the README file from the commit. This is auto-generated and should not be commited to git</li>
<li>You mentioned that your work is based on OpenIMScore. Do you actually use code from this project in your implementation? Then you need to add proper copyright notices that refer to this code.</li>
<li>With regards to copyright - you need to also choose a copyright to your code. Most modules choose "GPLv2 and later", but BSD would be also ok if you prefer this. Have a look to other modules for an  example copyright header.</li>
</ul>
<p>Best regards,<br>
Henning</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/kamailio/kamailio/pull/1605#issuecomment-408609505">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZRu0uun-X0M8cgzuwGmDBXRR4rArks5uLG03gaJpZM4VjXyg">mute the thread</a>.<img src="https://github.com/notifications/beacon/AF36ZS_sAhU_CCUGYxCG91vPLEsqY1GJks5uLG03gaJpZM4VjXyg.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/kamailio/kamailio","title":"kamailio/kamailio","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/kamailio/kamailio"}},"updates":{"snippets":[{"icon":"PERSON","message":"@henningw in #1605: Hello,\r\nthank you for your contribution! I did a quick look to the code and noticed a few simple areas that should refined before going looking deeper in the functionality.\r\n\r\n* Please remove the README file from the commit. This is auto-generated and should not be commited to git\r\n* You mentioned that your work is based on OpenIMScore. Do you actually use code from this project in your implementation? Then you need to add proper copyright notices that refer to this code.\r\n* With regards to copyright - you need to also choose a copyright to your code. Most modules choose \"GPLv2 and later\", but BSD would be also ok if you prefer this. Have a look to other modules for an  example copyright header.\r\n\r\nBest regards,\r\nHenning"}],"action":{"name":"View Pull Request","url":"https://github.com/kamailio/kamailio/pull/1605#issuecomment-408609505"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/kamailio/kamailio/pull/1605#issuecomment-408609505",
"url": "https://github.com/kamailio/kamailio/pull/1605#issuecomment-408609505",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [kamailio/kamailio] ims_ipsec_pcscf: sec-agree implementation for IMS (#1605)",
"sections": [
{
"text": "",
"activityTitle": "**Henning Westerholt**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@henningw",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"kamailio/kamailio\",\n\"issueId\": 1605,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close pull request",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"PullRequestClose\",\n\"repositoryFullName\": \"kamailio/kamailio\",\n\"pullRequestId\": 1605\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/kamailio/kamailio/pull/1605#issuecomment-408609505"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 361594016\n}"
}
],
"themeColor": "26292E"
}
]</script>