[GitHub] mike-jumper commented on issue #340: GUACAMOLE-660: Add Urdu Language Support
mike-jumper commented on issue #340: GUACAMOLE-660: Add Urdu Language Support URL: https://github.com/apache/guacamole-client/pull/340#issuecomment-453867532 OK - things look generally good code-wise, but I'm seeing a few issues when testing this new support. Most appear to be unexpected results of RTL text. 1. The icon indicating a field used for searching/filtering is still left-justified despite RTL text direction. ![filter-icon-left-side-1](https://user-images.githubusercontent.com/4632905/51090828-1d13fc80-1737-11e9-9015-9f5443eed486.png) ![filter-icon-left-side-2](https://user-images.githubusercontent.com/4632905/51090824-1c7b6600-1737-11e9-97e3-4ccae070512c.png) 2. The text of the connection status dialog is still left-justified despite RTL text direction. ![connection-status-dialog-text-left-justified](https://user-images.githubusercontent.com/4632905/51090823-1c7b6600-1737-11e9-9a12-dc88ea0c5520.png) 3. When Urdu is selected, the pagination buttons point weirdly inward. ![pagination-buttons-point-weirdly](https://user-images.githubusercontent.com/4632905/51090825-1c7b6600-1737-11e9-861b-c27eac41bd26.png) 4. Connection duration is still in English (and backwards due to RTL). ![durations-in-english](https://user-images.githubusercontent.com/4632905/51090826-1c7b6600-1737-11e9-9f87-3a318c463506.png) 5. Connection lists are still left-justified despite RTL text direction, and the expand button and tree lines for connection hierarchies is misaligned. ![expand-button-tree-lines-misaligned](https://user-images.githubusercontent.com/4632905/51090827-1d13fc80-1737-11e9-84a6-8dac05517ddf.png) 6. The menu icons are on the left side of the text despite RTL text direction. ![menu-icons-left-side](https://user-images.githubusercontent.com/4632905/51090829-1d13fc80-1737-11e9-9f91-bae99a5694b8.png) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] necouchman closed pull request #356: GUACAMOLE-699: Add helper script for verifying translations.
necouchman closed pull request #356: GUACAMOLE-699: Add helper script for verifying translations. URL: https://github.com/apache/guacamole-client/pull/356 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, comm...@guacamole.apache.org This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
Re: [DISCUSS] Migration of git repositories to gitbox (Was: Re: [NOTICE] Mandatory migration of git repositories to gitbox.apache.org)
So far, so good. We can merge with the merge button! Beware that GitHub now apparently provides two fields for the merge commit message. This threw me at first (isn't there only one message for a commit?) but it's really all for the same message. The first field is the first line, while the second, multi-line field is for an optional extended description to be included after a blank line in the same message. There seems to be some issues with the way emails, etc. work: 1) Once merged, the email from GitBox alerting to the merge introduces/complains about GitHub behavior twice. Based on how it's worded, it looks like this was two attempts to say the same thing, only one of which was meant to be kept: "[absolute ton of blank lines] This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): [the diff]" 2) GitBox automatically comments on the associated JIRA issue with an incorrect / partly useless comment saying that the committer that merged the PR commented on the PR: "mike-jumper commented on pull request #351: GUACAMOLE-683: Add OpenID support in Docker Build Scripts URL: https://github.com/apache/guacamole-client/pull/351 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org" (see https://issues.apache.org/jira/browse/GUACAMOLE-683?focusedCommentId=16741527=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16741527 ) It'd be nice if GitHub integration for JIRA were installed rather than having a bot comment on issues. The GitHub integration maintains a nice, unobtrusive list of relevant commits and the status of associated pull requests. I'm not sure why it's necessary to have an in-house bot comment and create links when a polished integration for JIRA already exists. 3) No emails containing the actual commits are sent out to commits@. There is an email regarding the pull request closure that goes to commits@ and others for the various commits, but those only the diff statistics and commit message, not the diff/commit itself. There is one message that does contain the entire diff (the one mentioned in #1 above), but it goes to dev@ and contains the entire diff. Whatever happened to the old functionality where there was one email per commit sent to commits@? Doesn't git provide its own commit to email mapping out-of-the-box? I'd think that there would be email-formatted diffs for the new commits sent to commits@, and a simple message (no diff) noting the fact that PR was merged sent to dev@. - Mike On Sun, Jan 13, 2019 at 1:22 AM Jean-Baptiste Onofré wrote: > Done, the repositories are now on gitbox. > > You just have to update the git remote. > > Regards > JB > > On 12/01/2019 21:11, Nick Couchman wrote: > > On Fri, Jan 11, 2019 at 1:44 AM Mike Jumper wrote: > > > >> On Thu, Jan 10, 2019, 22:29 Jean-Baptiste Onofré wrote: > >> > >>> Hi, > >>> > >>> No problem to do it now. It's very fast (let's 5 minutes) and smooth ;) > >>> > >>> Do you want me to create a Jira for INFRA ? > >>> > >> > >> Sure! Thanks, JB. > >> > >> - Mike > >> > > > > Thanks for doing this, JB. Any word on the timeline for it (maybe it's > > already done)? > > > > -Nick > > > > -- > Jean-Baptiste Onofré > jbono...@apache.org > http://blog.nanthrax.net > Talend - http://www.talend.com >
[GitHub] mike-jumper closed pull request #351: GUACAMOLE-683: Add OpenID support in Docker Build Scripts
mike-jumper closed pull request #351: GUACAMOLE-683: Add OpenID support in Docker Build Scripts URL: https://github.com/apache/guacamole-client/pull/351 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/guacamole-docker/bin/build-guacamole.sh b/guacamole-docker/bin/build-guacamole.sh index f72fb673d..c5ff255d4 100755 --- a/guacamole-docker/bin/build-guacamole.sh +++ b/guacamole-docker/bin/build-guacamole.sh @@ -126,3 +126,11 @@ if [ -f extensions/guacamole-auth-radius/target/guacamole-auth-radius*.jar ]; th mkdir -p "$DESTINATION/radius" cp extensions/guacamole-auth-radius/target/guacamole-auth-radius*.jar "$DESTINATION/radius" fi + +# Copy OPENID auth extension and schema modifications +# + +if [ -f extensions/guacamole-auth-openid/target/guacamole-auth-openid*.jar ]; then +mkdir -p "$DESTINATION/openid" +cp extensions/guacamole-auth-openid/target/guacamole-auth-openid*.jar "$DESTINATION/openid" +fi diff --git a/guacamole-docker/bin/start.sh b/guacamole-docker/bin/start.sh index 0732283a7..2ed50a929 100755 --- a/guacamole-docker/bin/start.sh +++ b/guacamole-docker/bin/start.sh @@ -404,6 +404,62 @@ END ln -s /opt/guacamole/radius/guacamole-auth-*.jar "$GUACAMOLE_EXT" } +## Adds properties to guacamole.properties which select the OPENID +## authentication provider, and configure it to connect to the specified OPENID +## provider. +## +associate_openid() { + +# Verify required parameters are present +if [ -z "$OPENID_AUTHORIZATION_ENDPOINT" ] || \ + [ -z "$OPENID_JWKS_ENDPOINT" ] || \ + [ -z "$OPENID_ISSUER" ] || \ + [ -z "$OPENID_CLIENT_ID" ] || \ + [ -z "$OPENID_REDIRECT_URI" ] +then +cat <
Re: [DISCUSS] Migration of git repositories to gitbox (Was: Re: [NOTICE] Mandatory migration of git repositories to gitbox.apache.org)
Done, the repositories are now on gitbox. You just have to update the git remote. Regards JB On 12/01/2019 21:11, Nick Couchman wrote: > On Fri, Jan 11, 2019 at 1:44 AM Mike Jumper wrote: > >> On Thu, Jan 10, 2019, 22:29 Jean-Baptiste Onofré > >>> Hi, >>> >>> No problem to do it now. It's very fast (let's 5 minutes) and smooth ;) >>> >>> Do you want me to create a Jira for INFRA ? >>> >> >> Sure! Thanks, JB. >> >> - Mike >> > > Thanks for doing this, JB. Any word on the timeline for it (maybe it's > already done)? > > -Nick > -- Jean-Baptiste Onofré jbono...@apache.org http://blog.nanthrax.net Talend - http://www.talend.com