Re: [GNAP] GNAP Editors' Use of GitHub Issues

Yaron Sheffer <yaronf.ietf@gmail.com> Wed, 25 November 2020 18:27 UTC

Return-Path: <yaronf.ietf@gmail.com>
X-Original-To: txauth@ietfa.amsl.com
Delivered-To: txauth@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id CD9533A1537 for <txauth@ietfa.amsl.com>; Wed, 25 Nov 2020 10:27:48 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.096
X-Spam-Level:
X-Spam-Status: No, score=-2.096 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, MIME_QP_LONG_LINE=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id d0sC--ZMU-mS for <txauth@ietfa.amsl.com>; Wed, 25 Nov 2020 10:27:44 -0800 (PST)
Received: from mail-ed1-x529.google.com (mail-ed1-x529.google.com [IPv6:2a00:1450:4864:20::529]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 5960A3A14FF for <txauth@ietf.org>; Wed, 25 Nov 2020 10:27:44 -0800 (PST)
Received: by mail-ed1-x529.google.com with SMTP id q16so3557339edv.10 for <txauth@ietf.org>; Wed, 25 Nov 2020 10:27:44 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=user-agent:date:subject:from:to:message-id:thread-topic:references :in-reply-to:mime-version; bh=IspZN1q2osmnlY5q72FJRN04XA6r1ALdL46bNTBTDnc=; b=ZIWjDmVFbIPb3TG4fX6g3cs0yMskb1igh4FuAQDVqPtOc1ejz7RF5vKwctrrZf8HJc DxDLeWy6INPqM2G4VCvEpMyOjMqjM4USe9k4T79B8r6TNYBVMOFHT8uwSyHpnO8gejE6 12C8QffUlSTIGZyiDHZmA26YUEjKRe78bvkR925BWdHzqWTviIJaKfvlCzNe3OhbB9mX m3S57tUrjO+FewPyA3d4WpaNFWnkXdGRq9w+p4xXiy4btWaUzblsjs4zYyAHHteH2BJu Y9Jvw7CdMUT0tsm3EHC1ZfZM+pbh1jy6PGzPqj4VvADUEQ97njJUtKEsipSoi7uSCJw/ y/qA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:user-agent:date:subject:from:to:message-id :thread-topic:references:in-reply-to:mime-version; bh=IspZN1q2osmnlY5q72FJRN04XA6r1ALdL46bNTBTDnc=; b=qt2ywInVFkGuRl70sSId93/vcK1chJXCoPizonRgkQil08ZM2sN26CRvqODltFMGxl CdwhiOGWnqEV1etEwKic9ULutZ11hYAjoGnzMVk+rWXjE5XfY2m/4VOF7J/nolTmnWX4 rV/+XF8rWG+kqFAIqxD4oWKPpRB7uETydscT3NUoMZGSrwq6/67TXz2FeKNHmbw9sRyK EVuNAYdTKHmaFRT4pDfFNplxwZDg7mUDOFl/7riY3Zq5/M3W/G1jsQnSztmOMzLrBUyG PlZwqJYUtYnHQsYzJJpTrA8YuIm9ulJWVEQsRRx4SvJK8OiRPw/nlNT5P7rT0B6oV0Ui 8McQ==
X-Gm-Message-State: AOAM532Tpw/SBd+fLgfqptHMWF0E+/5RqQTM2wM22XiaKGxb4L+ZEeg0 jgiffqOIqllQUP0p2C0gZvgmzcjs6b4lWg==
X-Google-Smtp-Source: ABdhPJzdup5CZ9gFeQBll/J3qePSmNbCkvgwwfqUdWrQIPqJUdPF57DqUnGxVmw2CqTiGK4fgoFtPg==
X-Received: by 2002:a50:9b5c:: with SMTP id a28mr4767452edj.139.1606328862579; Wed, 25 Nov 2020 10:27:42 -0800 (PST)
Received: from [172.26.49.35] (pub-corp-42-8.intuit.com. [91.102.42.8]) by smtp.gmail.com with ESMTPSA id rs27sm1753271ejb.34.2020.11.25.10.27.41 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Wed, 25 Nov 2020 10:27:42 -0800 (PST)
User-Agent: Microsoft-MacOutlook/16.43.20110804
Date: Wed, 25 Nov 2020 20:27:40 +0200
From: Yaron Sheffer <yaronf.ietf@gmail.com>
To: Aaron Parecki <aaron@parecki.com>, GNAP Mailing List <txauth@ietf.org>
Message-ID: <80EEA99F-3BCD-4FE9-9441-6D5EF4A606D9@gmail.com>
Thread-Topic: [GNAP] GNAP Editors' Use of GitHub Issues
References: <CAGBSGjqi8FdQg6RS_tpbW-R1JzeWiwKnJ2ObVxwMOkAbHaaJ_Q@mail.gmail.com>
In-Reply-To: <CAGBSGjqi8FdQg6RS_tpbW-R1JzeWiwKnJ2ObVxwMOkAbHaaJ_Q@mail.gmail.com>
Mime-version: 1.0
Content-type: multipart/alternative; boundary="B_3689180861_211338390"
Archived-At: <https://mailarchive.ietf.org/arch/msg/txauth/VkTnr203MFGiWP_2MjrWXqAI-GQ>
Subject: Re: [GNAP] GNAP Editors' Use of GitHub Issues
X-BeenThere: txauth@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: GNAP <txauth.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/txauth>, <mailto:txauth-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/txauth/>
List-Post: <mailto:txauth@ietf.org>
List-Help: <mailto:txauth-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/txauth>, <mailto:txauth-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 25 Nov 2020 18:27:49 -0000

Commenting on the proposed process (chair hat off):

 

I think “postponed” issues should not be closed. Once something is closed, we should be reasonably confident that it is resolved for good. People rarely search through closed issues.

 

Moreover, IMO the label “postponed” is not actionable. Instead, I suggest to mark such issues with future milestones, e.g. “IETF110”, meaning that at this time we will review the issue. Otherwise, the “postponed” issues will probably suffer the destiny of all “low priority” issues – they will be ignored for months and eventually closed en masse. See [1] for GitHub milestones.

 

Otherwise I am fine with the rest of the process.

 

Thanks,

                Yaron

 

[1] https://docs.github.com/en/free-pro-team@latest/github/managing-your-work-on-github/about-milestones

 

 

From: TXAuth <txauth-bounces@ietf.org> on behalf of Aaron Parecki <aaron@parecki.com>
Date: Wednesday, November 25, 2020 at 18:37
To: GNAP Mailing List <txauth@ietf.org>
Subject: [GNAP] GNAP Editors' Use of GitHub Issues

 

The editors met yesterday to discuss the issues that were pulled out of the previous draft text and document a process for how to resolve these and future issues. We would like to explain how we plan on using labels on GitHub issues to keep track of discussions and keep things moving.

When there are substantive issues or pull requests, the editors will avoid merging or closing those outright, and instead mark them as "pending", so that these can be brought to the attention of the larger group. If no additional discussion happens on these, the merge or close action will be taken in 7 days. Note for this first round we are setting the deadline for the issues below as Dec 11th due to the US holiday and the fact that this is the first time using this process.

"Pending Merge"
When specific text is proposed in a PR (by anyone, not limited to the editors), and the editors believe this text reflects the consensus of the working group, this marks that the PR will be merged in 7 days unless there is a clear alternative proposal accepted by the working group.

"Pending Close"
When the editors believe an issue no longer needs discussion, we'll mark it "Pending Close". The issue will be closed in 7 days unless someone brings new information to the discussion. This tag is not applied to issues that will be closed by a specific pull request.

There are two additional labels we will use to flag issues to the group.

"Needs Text"
The editors suggest this issue needs additional text in the spec to clarify why this section is needed and under what circumstances. Without a concrete proposal of text to be included in the spec, this section will be removed in a future update.

"Postponed"
This issue can be reconsidered in the future with a more concrete discussion but is not targeted for immediate concrete changes to the spec text. When used on its own, this label does not indicate that an issue is targeted to be closed. An issue may also be marked "Pending Close", and this is used so that we can distinguish closed issues between discussions that have concluded or things that we may want to revisit in the future. Remember that closed issues are not deleted and their contents are still findable and readable, and that new issues can reference closed issues.

With these labels in mind, here are the list of issues and their statuses we were able to discuss on our last editor's call. The action on these pending issues will be taken on Dec 11th to give the group enough time to review this list. For this first round, many of the issues are marked "Pending Close" as we're looking for low hanging fruit to prune the list of issues down. In the future, you can expect to see more "Pending Merge" issues as we're bringing proposed text to review by the WG.

Postponed:

* Generic claim extension mechanism
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/131

Pending Merge:

* Make access token mandatory for continuation API calls
** https://github.com/ietf-wg-gnap/gnap-core-protocol/pull/129

Postponed and Pending Close:

* Fetchable Keys
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/47
* Including OpenID Connect Claims
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/64
* Application communication with back-end
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/82
* Additional post-interaction protocols
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/83

Pending Close:
    
* HTTP PUT vs POST for rotating access tokens
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/100
* Use of hash with unique callback URL
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/84
* Interaction considerations
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/81
* Expanding dynamic reference handles
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/76
* Post interaction callback nonce
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/73
* Unique callback URIs 
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/55
* Instance identifier
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/46
* Requesting resources by reference
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/36
* Mapping resource references
** https://github.com/ietf-wg-gnap/gnap-core-protocol/issues/35

---

Aaron Parecki

https://aaronparecki.com

 

-- TXAuth mailing list TXAuth@ietf.org https://www.ietf.org/mailman/listinfo/txauth