profile
viewpoint
Jordi Olivares Provencio jordiolivares Barcelona

jordiolivares/Olipressor 1

High lossy compression algorithm for images

jordiolivares/AdvancedFizzBuzz-ML 0

Very complex and machine learned FizzBuzz

jordiolivares/battery-trigger 0

A small utility to execute commands when triggered by battery events

jordiolivares/berglas 0

A tool for managing secrets on Google Cloud

jordiolivares/cloud-builders-community 0

Community-contributed images for Google Cloud Build

jordiolivares/csv2gpx 0

A CSV conversion system to GPX for a custom CSV file format

jordiolivares/dkregistry-rs 0

A pure-Rust asynchronous library for Docker Registry API v2

startedGoogleCloudPlatform/berglas

started time in 24 days

push eventjordiolivares/dkregistry-rs

Stefan Junker

commit sha 0ede6720a3f52949ec6e206afd0a65510c18c9fc

auth: support anonymous authentication Allow authentication without credentials and add a test against the docker.io repo Co-authored-by: Stefan Junker <steveeJ@users.noreply.github.com>

view details

push time in a month

Pull request review commentcamallo/dkregistry-rs

Allow authentication without credentials for docker.io repo

 impl Client {             authentication_header,         )? {             WwwAuthenticateHeaderContent::Basic(_) => {-                let basic_auth = BasicAuth {-                    user: credentials.0,-                    password: Some(credentials.1),+                let basic_auth = {+                    let (user, password) =+                        credentials.expect("cannot authenticate without credentials");

I've accepted the proposed change by @steveeJ.

jordiolivares

comment created time in a month

push eventjordiolivares/dkregistry-rs

Jordi Olivares Provencio

commit sha a81a0190a3d65a4daea3dbd7a750e0ab984ba6b1

Update tests/net/docker_io/mod.rs Co-authored-by: Stefan Junker <steveeJ@users.noreply.github.com>

view details

push time in a month

Pull request review commentcamallo/dkregistry-rs

Allow authentication without credentials for docker.io repo

 fn test_dockerio_insecure() {     let res = runtime.block_on(futcheck).unwrap();     assert_eq!(res, true); }++#[test]+fn test_dockerio_unauthed() {

Yeah, it's a much better name

jordiolivares

comment created time in a month

push eventjordiolivares/dkregistry-rs

Jordi Olivares Provencio

commit sha d7537e7ea177c597099381d349b8628fbedd8c76

Update src/v2/auth.rs Co-authored-by: Stefan Junker <steveeJ@users.noreply.github.com>

view details

push time in a month

Pull request review commentcamallo/dkregistry-rs

Allow authentication without credentials for docker.io repo

 impl Client {             authentication_header,         )? {             WwwAuthenticateHeaderContent::Basic(_) => {-                let basic_auth = BasicAuth {-                    user: credentials.0,-                    password: Some(credentials.1),+                let basic_auth = {+                    let (user, password) =+                        credentials.expect("cannot authenticate without credentials");+                    BasicAuth {+                        user,+                        password: Some(password),+                    }                 };

Thanks, I'm still getting the ropes of Rust coming in from the C++/Scala world

jordiolivares

comment created time in a month

PR opened camallo/dkregistry-rs

Allow authentication without credentials for docker.io repo

Before the refactor of the auth logic (5a4ec71136d3c70d3675f11719a0788cb28eea7f) we could pull from docker.io without credentials.

After the commit that behaviour has changed and we unfortunately need to provide valid credentials as otherwise we can't pull from Docker Hub.

This PR adds back support for pulling from Docker Hub

+42 -14

0 comment

2 changed files

pr created time in a month

push eventjordiolivares/dkregistry-rs

Jordi Olivares Provencio

commit sha 142b15b48ea9844778fac679c37d6a2914427a76

Allow authentication without credentials for docker.io repo

view details

push time in a month

fork jordiolivares/dkregistry-rs

A pure-Rust asynchronous library for Docker Registry API v2

fork in a month

push eventjordiolivares/scalatags

Jordi Olivares Provencio

commit sha 334bd898e04c9994ff0e464a0a85fe236b649f97

Create new Escape function for strings for a ~2-3x improvement over the previous function.

view details

push time in 2 months

PR opened lihaoyi/scalatags

Create new Escape function for strings for a ~2-3x improvement

I was making some profiling for some code and the Escaping.escape function was a bottleneck. After making some implementation alternatives and benchmarking them this is the best function that's still maintainable.

Running jmh with -i5 -wi5 -f4 -t4 (5 warm-up iterations, 5 iterations, 4 JVM forks, and 4 threads) I obtained the following results:

[info] Benchmark                             Mode  Cnt        Score        Error  Units
[info] EscapeCharsBenchmark.newEscapeBench  thrpt   20  1318134.708 ± 166014.699  ops/s
[info] EscapeCharsBenchmark.oldEscape       thrpt   20   584732.517 ±  47941.948  ops/s
+34 -16

0 comment

1 changed file

pr created time in 2 months

push eventjordiolivares/scalatags

Jordi Olivares Provencio

commit sha 26619ecd5e81a3058ae36ae019109ed130aca780

Create new Escape function for strings for a ~2-3x improvement over the previous function.

view details

push time in 2 months

push eventjordiolivares/scalatags

Jordi Olivares Provencio

commit sha c299a976417b7743193e86aaa850ea66ee682fa5

Create new Escape function for strings for a ~2-3x improvement over the previous function.

view details

push time in 2 months

push eventjordiolivares/scalatags

Jordi Olivares Provencio

commit sha 3fa51aef6a611d7049ceab0d56c0f8951ed60a22

Create new Escape function for strings for a ~2-3x improvement over the previous function.

view details

push time in 2 months

push eventjordiolivares/scalatags

Jordi Olivares Provencio

commit sha 72c0d0a3632d765c5d7f616380abde6ad4d9e3fa

Create new Escape function for strings for a ~2-3x improvement over the previous function.

view details

push time in 2 months

PR opened softwaremill/sttp

Make OkHttpBackend compatible with Java 8 again

With the rewrite for using an InputStream the OkHttp backend used the InputStream.readAllBytes() which doesn't exist in Java 8.

Unfortunately for many users, Java 8 is still the norm, so this should make the library compatible with them again.

+1 -1

0 comment

1 changed file

pr created time in 2 months

push eventjordiolivares/sttp

Jordi Olivares

commit sha db4718a5c837f7ac02b090de0c5cabee12d7d1da

Make OkHttp compatible with Java 8 again

view details

push time in 2 months

push eventjordiolivares/sttp

Jordi Olivares

commit sha 604bd0fea5d785f782ae500e45e64db21eba10cb

Make OkHttp compatible with Java 8 again

view details

push time in 2 months

fork jordiolivares/sttp

The Scala HTTP client you always wanted!

https://softwaremill.com/open-source/

fork in 2 months

push eventjordiolivares/scalatags

Jordi Olivares Provencio

commit sha aa41f5d5663aba8fcc02f117d947677f5da8cacd

Create new Escape function for strings for a 2x improvement

view details

push time in 2 months

fork jordiolivares/scalatags

ScalaTags is a small XML/HTML construction library for Scala.

fork in 2 months

startedTechEmpower/FrameworkBenchmarks

started time in 2 months

more