Cody Cutrer
ccutrer

Organizations

@instructure
Aug 26, 2016
@ccutrer
Aug 26, 2016
@ccutrer
Aug 22, 2016
ccutrer commented on pull request instructure/canvas-lms#879
@ccutrer

are you planning on doing the gem upgrades in a separate PR?

Aug 22, 2016
ccutrer commented on pull request instructure/canvas-lms#877
@ccutrer

while you're in the area - can you confirm that this case will never happen again (I'm pretty sure it was only for Ruby 1.9, which Canvas no longer…

Aug 19, 2016
ccutrer commented on commit simonista/open4@fffed425c2
@ccutrer

The comment has since been removed.

Aug 17, 2016
@ccutrer
ccutrer deleted branch origin/master at instructure/ruby-saml
Aug 4, 2016
Aug 4, 2016
@ccutrer
Aug 4, 2016
Jul 11, 2016
ccutrer commented on issue instructure/canvas-lms#854
@ccutrer

Is this a repeating error, or only happened once? IIRC, we removed a periodic job, which due to how they are scheduled would cause this error to ha…

Jun 23, 2016
@ccutrer
  • @ccutrer b16736e
    AR 5.0.0rc2 compatibility
Jun 23, 2016
ccutrer opened pull request instructure/shackles#14
@ccutrer
thread safety
1 commit with 32 additions and 3 deletions
Jun 23, 2016
Jun 23, 2016
@ccutrer
Jun 20, 2016
ccutrer commented on issue instructure/canvas-lms#827
@ccutrer

if you gem install the same version of thrift that the Gemfile specifies (0.8.0), then bundler will use the version installed by gem install.

Jun 10, 2016
ccutrer commented on issue instructure/canvas-lms#842
@ccutrer

If you created the user via API, you likely have permission to masquerade as them. So you use the API with your own credentials, and then pass the …

Jun 2, 2016
ccutrer commented on issue instructure/canvas-lms#836
@ccutrer

Yes, I am absolutely opposed to purposely over-encoding something just to munge it later. In this case I can't find a counter-example, but taking a…

Jun 2, 2016
@ccutrer
Jun 2, 2016
@ccutrer
Jun 2, 2016
ccutrer commented on issue instructure/canvas-lms#836
@ccutrer

Yes, pretty much any reserved character will have problems: irb(main):002:0> URI.encode(URI.decode('https://example.com/a%3fb')) => "https://exampl…

Jun 1, 2016
ccutrer commented on issue instructure/canvas-lms#836
@ccutrer

Yes: irb(main):003:0> URI.encode(URI.decode('http://example.com/a%2fb')) => "http://example.com/a/b" The slash that was previously percent encoded…

Jun 1, 2016
ccutrer commented on issue instructure/canvas-lms#837
@ccutrer

I've already pushed up a fix internally, and it's been code reviewed, but not QA'd yet. Hopefully that happens in the next day or two, and this wil…

Jun 1, 2016
ccutrer commented on issue instructure/canvas-lms#836
@ccutrer

That constant comes from section 2.2 of https://www.ietf.org/rfc/rfc2396.txt. Looking at the method in more detail, escape by default escapes UNSAF…

May 31, 2016
ccutrer commented on issue instructure/canvas-lms#836
@ccutrer

Naively encoding a full URL is not a good solution. For example, if I encode https://somesite.com/page?stuff=stuff&, the ampersand would get encode…