Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Form] Switched to correct gender of "Token" #9686

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

Closed
wants to merge 1 commit into from
Closed

[Form] Switched to correct gender of "Token" #9686

wants to merge 1 commit into from

Conversation

ChristianGaertner
Copy link
Contributor

The neutral gender "das Token" is used in science to describe an array of bits or chars, where as the masculin "der Token" is commenly used for keys in communications and authentication in the network.

I guess the a CSRF-Token is more a key for authentication then a bit array.

Cheers,

Christian

Q A
Bug fix? yes/kinda
New feature? not really
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets ~
Doc PR none

The neutral gender "das Token" is used in science to describe an array of bits or chars, where as the masculin "der Token" is commenly used for keys in communications and authentication in the network.

I guess the a CSRF-Token is more a key for authentication then a bit array.
@ChristianGaertner
Copy link
Contributor Author

NOTE: The build failed in the StopWatch Component. Not the Form component.

@cordoval
Copy link
Contributor

cordoval commented Dec 3, 2013

yes @ChristianGaertner it has been happening often, please go to travis and relaunch your run from there so it can be green and all good

@wouterj
Copy link
Member

wouterj commented Dec 3, 2013

@cordoval only contributor access people can relaunch the run afaik, so you have to do this

@cordoval
Copy link
Contributor

cordoval commented Dec 3, 2013

no @wouterj is his fork

@cordoval
Copy link
Contributor

cordoval commented Dec 3, 2013

but in any case i jsut relaunched it 👶

@ChristianGaertner
Copy link
Contributor Author

Thanks for triggering the rebuild!

This time a failure in the Process component. Anyone a clue?! I mean it's for sure not caused by this commit (as the previous build passed with the PHP version which just errored...)

@wouterj
Copy link
Member

wouterj commented Dec 3, 2013

don't worry about the build failure, @fabpot will discover it's not caused by this PR.

fabpot added a commit that referenced this pull request Dec 4, 2013
…ertner)

This PR was submitted for the 2.2 branch but it was merged into the 2.3 branch instead (closes #9686).

Discussion
----------

[Form] Switched to correct gender of "Token"

The neutral gender "das Token" is used in science to describe an array of bits or chars, where as the masculin "der Token" is commenly used for keys in communications and authentication in the network.

I guess the a CSRF-Token is more a key for authentication then a bit array.

Cheers,

Christian

| Q             | A
| ------------- | ---
| Bug fix?      | yes/kinda
| New feature?  | not really
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | ~
| Doc PR        | none

Commits
-------

c2bc9ae Switched to correct gender of "Token"
@fabpot fabpot closed this Dec 5, 2013
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.

4 participants