Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Allow Connect API Key#133

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
schloerke merged 17 commits intorstudio:masterfromalexkgold:connect_api_key
Feb 9, 2021
Merged

Conversation

@alexkgold
Copy link

@alexkgoldalexkgold commentedSep 10, 2020
edited by schloerke
Loading

@CLAassistant
Copy link

CLAassistant commentedSep 10, 2020
edited
Loading

CLA assistant check
All committers have signed the CLA.

@alexkgold
Copy link
Author

Was just demoing this to the SE team -- one thing popped up -- RStudio Connect provides 2 URLs for each content. There's the one that comes up in your URL bar when you navigate to the content inside the Connect dashboard, and the "content URL" for solo mode.

If you put the former intoshinyloadtest with an API key, the system just hangs indefinitely with no error.shinycannon does throw an error

CONNECT_API_KEY=T5opptrM4yXfOlFwDQwta4qjedebPsQB java -jar target/shinycannon-1.1.0-jar-with-dependencies.jar ../shinyloadtest/recording.log https://colorado.rstudio.com/rsc/connect/#/apps/7278zsh: bad pattern: https://colorado.rstudio.com/rsc/connect/#/apps/7278

But this error probably could be more friendly/helpful.

I think we should check for the form of the URL (look for a#?) and throw an error.

@schloerke -- see e.g.https://colorado.rstudio.com/rsc/connect/#/apps/7278/access vshttps://colorado.rstudio.com/rsc/content/7278 (I've given you access to the app on Colorado RSC if you want to take a look). Want me to make a PR here? I'm less confident of my ability to do so inshinycannon.

@schloerke
Copy link
Collaborator

@akgold Issue added:#135

Co-authored-by: Barret Schloerke <barret@rstudio.com>
@schloerkeschloerke merged commit007e16d intorstudio:masterFeb 9, 2021
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@schloerkeschloerkeschloerke approved these changes

@wchwchwch approved these changes

@jcheng5jcheng5jcheng5 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Throw error if RSC url has#

5 participants

@alexkgold@CLAassistant@schloerke@wch@jcheng5

[8]ページ先頭

©2009-2025 Movatter.jp