ABCDEFGHIJKLMNOPQRSTUVWXYZAA
1
No.CommentsTypeDispositionsFixedClosed
2
mbj1[mbj1]We should always use the term “OpenID Connect” and never abbreviate it to just “Connect”. It’s our brand – let’s use it correctly.edThere would be too much "OpenID" in the text. Define Connect OP instead.
Superseeded by #945
3
mbj2[mbj2]Put quotation marks around claims names. This should become “iss”/”sub”. This should be done for all other protocol literals as well.edAccept in principle. Potentially superseeded by #941. https://bitbucket.org/openid/connect/issue/941/06af06a
4
mbj3[mbj3]Titles need to be in title case, per English grammar rules. Thus, the P and T must be capitalized.edAccept in principle. The change should be applied to the main text instead. 2cccb6c
5
mbj4[mbj4]It’s inconsistent to use Relying Party and OP together. Spell both out.edAccept in principle. The change should be applied to the main text instead. 2cccb6c
6
mbj5[mbj5]This draft has lots of <title> tags without anchor members, even though I pointed this out in my -02 comments. Please fix all of these.edAccept. Note: they are not being used as reference so this is strictly editorial. b49f7f6
7
mbj6[mbj6]Put quotation marks around uses of protocol literalsedAccept in principle. These are to be dealt with <spanx> and the rendering would depend on the file format. 37ac30b
8
mbj7[mbj7]Title case needededAccept in principle. The change should be applied to the main text instead. 37ac30b
9
mbj8[mbj8]This construction is less awkward grammaticallyedAccept in principle. The change should be applied to the main text instead. 37ac30b
10
mbj9[mbj9]Title caseedAccept in principle. The change should be applied to the main text instead. 37ac30b
11
mbj10[mbj10]Title caseedAccept in principle. The change should be applied to the main text instead. 37ac30b
12
mbj11[mbj11]Title caseedAccept in principle. The change should be applied to the main text instead. 37ac30b
13
mbj12[mbj12]EOL normally stands for end-of-line. Don’t use abbreviations when they’re not universally understood.edAccept in principle. The change should be applied to the main text instead.

NOTE: Editor doubts the comment that EOL normally stands for end-of-line. Simple Google search of the EOL in incognito mode derives End-of-Life befor End-of-Line. End-of-Line only appears in the second page. 		37ac30b
14
mbj13[mbj13]Title caseedAccept in principle. The change should be applied to the main text instead. 37ac30b
15
mbj14[mbj14]Remove this duplicate section.edSuperseeded by mbj30-
16
mbj15[mbj15]This construction is more grammaticaledAccept in principle. The change should be applied to the main text instead. 37ac30b
17
mbj16[mbj16]Title caseedAccept in principle. The change should be applied to the main text instead. 37ac30b
18
mbj17[mbj17]This section duplicates the content of the Notices section, and is not in any other OpenID standards. Please remove it.edInvalid. This is put by the compiler unless the spec is specified as a private draft. Perhaps Appendix A should be removed or the template for HTML compilation should be fixed.
Editor regards the current use of the <?rfc private> directive in the existing specs XML is the bug, not this one. 		06af06a
19
mbj18[mbj18]OpenID Connect is not a version of OpenID 2.0edAccept Modified. Change it to new version of OpenID Authentication. aee5420
20
mbj19[mbj19]This construction is awkward.edReject. This is taken from ISO Guide 2. 1
21
mbj20[mbj20]You’re missing a space here. I pointed this out in my draft -02 comments but it wasn’t added to -03.edAccept. f9128f3
22
mbj21[mbj21]This is more grammaticaledAccept. 5487713
23
mbj22[mbj22]I would change this term to OpenID Connect OP so that we’re never using “Connect” without “OpenID”. Again, we should use our own trademarks correctly.edDiscuss. Connect is not a trademark but just points to one of our spec. Use of OpenID as suggested seem to make the spec have too many "OpenID" in it.
24
mbj23[mbj23]More preciseedAccept. 9c75054
25
mbj24[mbj24]Delete this extra spaceedAccept. 0569b6d
26
mbj25[mbj25]This should be a period since a list doesn’t immediately follow.edAccept. 3687621
27
mbj26[mbj26]Use spanx verb around all protocol literalsedAccept.
c4f7387 06af06a
28
mbj27[mbj27]Less awkwardteReject. We are dealing with the Authoritativeness of the service and not the identifier. See issue #937 as well. -
29
mbj28[mbj28]Plural usageedAccept. 8b9ed89
30
mbj29[mbj29]This is awkwardedAccept. 677e15c
31
mbj30[mbj30]Delete this duplicate, empty section!edAccept. 37ac30b
32
mbj31[mbj31]Plural usageedAccept. 320aed4
33
mbj32[mbj32]Add the missing space hereedAccept. 7003542
34
mbj33[mbj33]This was misspellededAccept. 6cd3336
35
mbj34[mbj34]MITRE is spelled in uppercaseedAccept. 6cd3336
36
mbj35[mbj35]This was misspellededAccept. eec6d5a
37
mbj36[mbj36]Delete these. This duplicates the contents of the Notices section and isn’t how we do this in any of the other OpenID Connect specs.edSuperseeded by #945-
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100