profile
viewpoint
Brendan Ashworth brendanashworth MIT Cambridge, Massachusetts

brendanashworth/generate-password 108

NodeJS library for generating cryptographically-secure passwords.

brendanashworth/bench 11

Benchmark tools for C

brendanashworth/go-ftp 11

Go FTP library, complete with an authentication API and easy to use documentation.

brendanashworth/flatfile 6

basic JSON-style flatfile storage

brendanashworth/fft-small 5

cooley-tukey 2-radix discrete fast fourier transform in C for embedded systems

brendanashworth/diskdrive 3

OS compatible bindings for various CD/DVD drive utilities with NodeJS (OSX + Linux supported)

brendanashworth/cli-table 1

Draw tables in a command line interface. Just like MySQL.

brendanashworth/CraftBukkit 1

The Minecraft Server Mod API Implementation

brendanashworth/accountant 0

PHP application for keeping tally of running balance

fork brendanashworth/gonum

Gonum is a set of numeric libraries for the Go programming language. It contains libraries for matrices, statistics, optimization, and more

https://www.gonum.org/

fork in 3 days

startedgonum/gonum

started time in 3 days

created tagbrendanashworth/generate-password

tagv1.5.1

NodeJS library for generating cryptographically-secure passwords.

created time in 8 days

PR closed brendanashworth/generate-password

Add lowercase property into Options

lowercase is missing in generate.d.ts

I added it. :)

Thanks!

+1 -0

3 comments

1 changed file

sunjoong85

pr closed time in 8 days

pull request commentbrendanashworth/generate-password

Add lowercase property into Options

Thank you @sunjoong85 and others for bumping me — I've merged this in https://github.com/brendanashworth/generate-password/commit/0eb0f498be34d98c72dd78d4423f9980ca83878a and it's now released as 1.5.1.

sunjoong85

comment created time in 8 days

push eventbrendanashworth/generate-password

Brendan Ashworth

commit sha 53f7b795e2861d97b9aef84bd4ecc85f83f20deb

publish 1.5.1

view details

push time in 8 days

push eventbrendanashworth/generate-password

sunjoong85

commit sha 0eb0f498be34d98c72dd78d4423f9980ca83878a

Add lowercase property into Options (#34)

view details

push time in 8 days

startedjbarlow83/OCRmyPDF

started time in 14 days

issue closedbrendanashworth/generate-password

Setting numbers to true in options does not guarantee number used

Details

I'm not sure if the boolean values were intended to always include that character type or only most of the time, but I used this library to aid in a temporary password flow and had a number of instances where providing the following options resulted in a password string without a number.

{
    length: 15,
    lowercase: true,
    uppercase: true,
    symbols: true,
    numbers: true,
    exclude: '+=',
}

I tried this with multiple other options configurations and found that passing a numbers value of true does not in fact guarantee that the generated string will include a numeric value.

closed time in 21 days

dallen4

issue commentbrendanashworth/generate-password

Setting numbers to true in options does not guarantee number used

@dallen4 it's true that the strict option is needed to ensure those characters are included in the password. While it may be less intuitive, having that as the default results in passwords with slightly more entropy = strength. Perhaps that should be better documented (PRs are welcome!), but I'm going to close this issue otherwise. 🍾

dallen4

comment created time in 21 days

PR closed georust/gpx

Small Parser Improvements

The parser failed to parse gpx files from gpsies.com. With these changes it is now possible to parse

  • link elements inside trk tags
  • extensions tag inside metadata tag

I also added a testcase.

+256 -0

3 comments

4 changed files

JDemler

pr closed time in a month

pull request commentgeorust/gpx

Small Parser Improvements

@JDemler thank you for putting this pull request together, sorry for taking forever to review it. I've merged it in https://github.com/georust/gpx/commit/5c484e972df9f539ff0ff23bb3cad4e5f47b2dee and will be releasing shortly.

The description to desc would be a breaking change, that might have been my mistake. Would you like to submit another PR for it? I think it can be released as a breaking change, say 1.0.0.

JDemler

comment created time in a month

push eventgeorust/gpx

Jakob Demler

commit sha 5c484e972df9f539ff0ff23bb3cad4e5f47b2dee

parse extensions in metadata, links in trk

view details

Brendan Ashworth

commit sha 2db345b59e6f73211be37287a3303aa76d255252

0.8.0 release

view details

push time in a month

PR closed brendanashworth/generate-password

Use easier `every` instead of `reduce`

There is no need to use Array.prototype.reduce over Array.prototype.every because no state needs to be maintained between iterations. This reduces complexity and also makes the abort line superfluous

+3 -6

1 comment

1 changed file

jonasholtkamp

pr closed time in a month

pull request commentbrendanashworth/generate-password

Use easier `every` instead of `reduce`

Thank you @jonasholtkamp , I've merged this in https://github.com/brendanashworth/generate-password/commit/25d2c8423e9ddf966599a99c58c3da8b37a091db and will be releasing it in the next patch release.

jonasholtkamp

comment created time in a month

push eventbrendanashworth/generate-password

Jonas Holtkamp

commit sha 25d2c8423e9ddf966599a99c58c3da8b37a091db

Use easier `every` instead of `reduce` (#32) There is no need to use `Array.prototype.reduce` over `Array.prototype.every` because no state needs to be maintained between iterations. This reduces complexity and also makes the abort line superfluous

view details

push time in a month

issue closedbrendanashworth/generate-password

The package is missing the required crypto dependency.

Can you update the NPM package to include the required dependencies, such as crypto?

closed time in a month

kahanu

issue commentbrendanashworth/generate-password

The package is missing the required crypto dependency.

I'm going to go ahead and close this issue as there isn't much to do here besides using the browser fork.

kahanu

comment created time in a month

issue closedbrendanashworth/generate-password

Use in angular project

Please update readme how can using angular and ionic project(examples)

closed time in a month

utpaul

issue commentbrendanashworth/generate-password

Use in angular project

I'm going to go ahead and close this issue as there isn't really much to do here, besides maybe using the browser fork.

utpaul

comment created time in a month

issue closedbrendanashworth/generate-password

Is it guaranteed that there will be at least one lowercase character when strict mode is true?

Is it guaranteed that there will be at least one lowercase character when multiple options, including strict, are set to true. For example, if my options object looks like

const options = {
	length: 4,
	numbers: true,
	symbols: true,
	uppercase: true,
	strict: true,
}

Is it possible that generator.generate(options) will return a string without any lowercase characters. That is, it will only contain numbers, symbols and uppercase characters.

It would be helpful to add the answer to this question to the README. Thanks!

closed time in a month

collin-pham

issue commentbrendanashworth/generate-password

Is it guaranteed that there will be at least one lowercase character when strict mode is true?

#31 has been merged and released into 1.5.0 so this should be fixed! https://github.com/brendanashworth/generate-password/commit/1862283ee922b359f6f8636909ee0b41e6fd3ce4

collin-pham

comment created time in a month

push eventbrendanashworth/generate-password

Brendan Ashworth

commit sha 1862283ee922b359f6f8636909ee0b41e6fd3ce4

publish 1.5.0

view details

push time in a month

PR closed brendanashworth/generate-password

Bump eslint from 2.13.1 to 4.18.2 dependencies

Bumps eslint from 2.13.1 to 4.18.2. <details> <summary>Release notes</summary>

Sourced from eslint's releases.

v4.18.2

  • 6b71fd0 Fix: table@4.0.2, because 4.0.3 needs "ajv": "^6.0.1" (#10022) (Mathieu Seiler)
  • 3c697de Chore: fix incorrect comment about linter.verify return value (#10030) (Teddy Katz)
  • 9df8653 Chore: refactor parser-loading out of linter.verify (#10028) (Teddy Katz)
  • f6901d0 Fix: remove catastrophic backtracking vulnerability (fixes #10002) (#10019) (Jamie Davis)
  • e4f52ce Chore: Simplify dataflow in linter.verify (#10020) (Teddy Katz)
  • 33177cd Chore: make library files non-executable (#10021) (Teddy Katz)
  • 558ccba Chore: refactor directive comment processing (#10007) (Teddy Katz)
  • 18e15d9 Chore: avoid useless catch clauses that just rethrow errors (#10010) (Teddy Katz)
  • a1c3759 Chore: refactor populating configs with defaults in linter (#10006) (Teddy Katz)
  • aea07dc Fix: Make max-len ignoreStrings ignore JSXText (fixes #9954) (#9985) (Rachael Sim)

v4.18.1

  • f417506 Fix: ensure no-await-in-loop reports the correct node (fixes #9992) (#9993) (Teddy Katz)
  • 3e99363 Docs: Fixed typo in key-spacing rule doc (#9987) (Jaid)
  • 7c2cd70 Docs: deprecate experimentalObjectRestSpread (#9986) (Toru Nagashima)

v4.18.0

  • 70f22f3 Chore: Apply memoization to config creation within glob utils (#9944) (Kenton Jacobsen)
  • 0e4ae22 Update: fix indent bug with binary operators/ignoredNodes (fixes #9882) (#9951) (Teddy Katz)
  • 47ac478 Update: add named imports and exports for object-curly-newline (#9876) (Nicholas Chua)
  • e8efdd0 Fix: support Rest/Spread Properties (fixes #9885) (#9943) (Toru Nagashima)
  • f012b8c Fix: support Async iteration (fixes #9891) (#9957) (Toru Nagashima)
  • 74fa253 Docs: Clarify no-mixed-operators options (fixes #9962) (#9964) (Ivan Hayes)
  • 426868f Docs: clean up key-spacing docs (fixes #9900) (#9963) (Abid Uzair)
  • 4a6f22e Update: support eslint-disable-* block comments (fixes #8781) (#9745) (Erin)
  • 777283b Docs: Propose fix typo for function (#9965) (John Eismeier)
  • bf3d494 Docs: Fix typo in max-len ignorePattern example. (#9956) (Tim Martin)
  • d64fbb4 Docs: fix typo in prefer-destructuring.md example (#9930) (Vse Mozhet Byt)
  • f8d343f Chore: Fix default issue template (#9946) (Kai Cataldo)

v4.17.0

  • 1da1ada Update: Add "multiline" type to padding-line-between-statements (#8668) (Matthew Bennett)
  • bb213dc Chore: Use messageIds in some of the core rules (#9648) (Jed Fox)
  • 1aa1970 Docs: remove outdated rule naming convention (#9925) (Teddy Katz)
  • 3afaff6 Docs: Add prefer-destructuring variable reassignment example (#9873) (LePirlouit)
  • d20f6b4 Fix: Typo in error message when running npm (#9866) (Maciej Kasprzyk)
  • 51ec6a7 Docs: Use GitHub Multiple PR/Issue templates (#9911) (Kai Cataldo)
  • dc80487 Update: space-unary-ops uses astUtils.canTokensBeAdjacent (fixes #9907) (#9906) (Kevin Partington)
  • 084351b Docs: Fix the messageId example (fixes #9889) (#9892) (Jed Fox)
  • 9cbb487 Docs: Mention the globals key in the no-undef docs (#9867) (Dan Dascalescu)

v4.16.0

  • e26a25f Update: allow continue instead of if wrap in guard-for-in (fixes #7567) (#9796) (Michael Ficarra)
  • af043eb Update: Add NewExpression support to comma-style (#9591) (Frazer McLean)
  • 4f898c7 Build: Fix JSDoc syntax errors (#9813) (Matija Marohnić)
  • 13bcf3c Fix: Removing curly quotes in no-eq-null report message (#9852) (Kevin Partington)
  • b96fb31 Docs: configuration hierarchy for CLIEngine options (fixes #9526) (#9855) (PiIsFour)
  • 8ccbdda Docs: Clarify that -c configs merge with .eslintrc.* (fixes #9535) (#9847) (Kevin Partington)
  • 978574f Docs: Fix examples for no-useless-escape (#9853) (Toru Kobayashi) </tr></table> ... (truncated) </details> <details> <summary>Changelog</summary>

Sourced from eslint's changelog.

v4.18.2 - March 2, 2018

  • 6b71fd0 Fix: table@4.0.2, because 4.0.3 needs "ajv": "^6.0.1" (#10022) (Mathieu Seiler)
  • 3c697de Chore: fix incorrect comment about linter.verify return value (#10030) (Teddy Katz)
  • 9df8653 Chore: refactor parser-loading out of linter.verify (#10028) (Teddy Katz)
  • f6901d0 Fix: remove catastrophic backtracking vulnerability (fixes #10002) (#10019) (Jamie Davis)
  • e4f52ce Chore: Simplify dataflow in linter.verify (#10020) (Teddy Katz)
  • 33177cd Chore: make library files non-executable (#10021) (Teddy Katz)
  • 558ccba Chore: refactor directive comment processing (#10007) (Teddy Katz)
  • 18e15d9 Chore: avoid useless catch clauses that just rethrow errors (#10010) (Teddy Katz)
  • a1c3759 Chore: refactor populating configs with defaults in linter (#10006) (Teddy Katz)
  • aea07dc Fix: Make max-len ignoreStrings ignore JSXText (fixes #9954) (#9985) (Rachael Sim)

v4.18.1 - February 20, 2018

  • f417506 Fix: ensure no-await-in-loop reports the correct node (fixes #9992) (#9993) (Teddy Katz)
  • 3e99363 Docs: Fixed typo in key-spacing rule doc (#9987) (Jaid)
  • 7c2cd70 Docs: deprecate experimentalObjectRestSpread (#9986) (Toru Nagashima)

v4.18.0 - February 16, 2018

  • 70f22f3 Chore: Apply memoization to config creation within glob utils (#9944) (Kenton Jacobsen)
  • 0e4ae22 Update: fix indent bug with binary operators/ignoredNodes (fixes #9882) (#9951) (Teddy Katz)
  • 47ac478 Update: add named imports and exports for object-curly-newline (#9876) (Nicholas Chua)
  • e8efdd0 Fix: support Rest/Spread Properties (fixes #9885) (#9943) (Toru Nagashima)
  • f012b8c Fix: support Async iteration (fixes #9891) (#9957) (Toru Nagashima)
  • 74fa253 Docs: Clarify no-mixed-operators options (fixes #9962) (#9964) (Ivan Hayes)
  • 426868f Docs: clean up key-spacing docs (fixes #9900) (#9963) (Abid Uzair)
  • 4a6f22e Update: support eslint-disable-* block comments (fixes #8781) (#9745) (Erin)
  • 777283b Docs: Propose fix typo for function (#9965) (John Eismeier)
  • bf3d494 Docs: Fix typo in max-len ignorePattern example. (#9956) (Tim Martin)
  • d64fbb4 Docs: fix typo in prefer-destructuring.md example (#9930) (Vse Mozhet Byt)
  • f8d343f Chore: Fix default issue template (#9946) (Kai Cataldo)

v4.17.0 - February 2, 2018

  • 1da1ada Update: Add "multiline" type to padding-line-between-statements (#8668) (Matthew Bennett)
  • bb213dc Chore: Use messageIds in some of the core rules (#9648) (Jed Fox)
  • 1aa1970 Docs: remove outdated rule naming convention (#9925) (Teddy Katz)
  • 3afaff6 Docs: Add prefer-destructuring variable reassignment example (#9873) (LePirlouit)
  • d20f6b4 Fix: Typo in error message when running npm (#9866) (Maciej Kasprzyk)
  • 51ec6a7 Docs: Use GitHub Multiple PR/Issue templates (#9911) (Kai Cataldo)
  • dc80487 Update: space-unary-ops uses astUtils.canTokensBeAdjacent (fixes #9907) (#9906) (Kevin Partington)
  • 084351b Docs: Fix the messageId example (fixes #9889) (#9892) (Jed Fox)
  • 9cbb487 Docs: Mention the globals key in the no-undef docs (#9867) (Dan Dascalescu)

v4.16.0 - January 19, 2018

  • e26a25f Update: allow continue instead of if wrap in guard-for-in (fixes #7567) (#9796) (Michael Ficarra)
  • af043eb Update: Add NewExpression support to comma-style (#9591) (Frazer McLean) </tr></table> ... (truncated) </details> <details> <summary>Commits</summary>
  • 22ff6f3 4.18.2
  • 817b84b Build: changelog update for 4.18.2
  • 6b71fd0 Fix: table@4.0.2, because 4.0.3 needs "ajv": "^6.0.1" (#10022)
  • 3c697de Chore: fix incorrect comment about linter.verify return value (#10030)
  • 9df8653 Chore: refactor parser-loading out of linter.verify (#10028)
  • f6901d0 Fix: remove catastrophic backtracking vulnerability (fixes #10002) (#10019)
  • e4f52ce Chore: Simplify dataflow in linter.verify (#10020)
  • 33177cd Chore: make library files non-executable (#10021)
  • 558ccba Chore: refactor directive comment processing (#10007)
  • 18e15d9 Chore: avoid useless catch clauses that just rethrow errors (#10010)
  • Additional commits viewable in compare view </details> <details> <summary>Maintainer changes</summary>

This version was pushed to npm by eslint, a new releaser for eslint since your current version. </details> <br />

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot ignore this [patch|minor|major] version will close this PR and stop Dependabot creating any more for this minor/major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

</details>

+1775 -3

1 comment

5 changed files

dependabot[bot]

pr closed time in a month

pull request commentbrendanashworth/generate-password

Document lowercase restriction, and add lowercase false option.

Thank you @donedgardo , I've pushed this in https://github.com/brendanashworth/generate-password/commit/960858c1cc7784b259c7d403ab5e26da7f5f32c0 and will be releasing it soon. Sorry about the delay.

donedgardo

comment created time in a month

push eventbrendanashworth/generate-password

Brendan Ashworth

commit sha 979fe302ac6b8edccdf3f37b9a8b952a7020df87

make lint pass, add npm versions, add package-lock

view details

push time in a month

push eventbrendanashworth/generate-password

Brendan Ashworth

commit sha 29bc562edfa7327817089a3c5f89f6d5fe090688

make lint pass, add npm versions, add package-lock

view details

push time in a month

push eventbrendanashworth/generate-password

Brendan Ashworth

commit sha 7b94cb88a46d444b2f041c7d33d5588c3a00db24

make lint pass, add npm versions, add package-lock

view details

push time in a month

push eventbrendanashworth/generate-password

Brendan Ashworth

commit sha 08b1d2a906065aaa2b0fc5f7e42b9f7d23a5ee24

make lint pass, add npm versions, add package-lock

view details

push time in a month

push eventbrendanashworth/generate-password

Brendan Ashworth

commit sha 64f70a08a46ae876f1e33292b8f24bcabb0beefb

make lint pass, add npm versions, add package-lock

view details

push time in a month

push eventbrendanashworth/generate-password

Edgardo Carreras

commit sha 960858c1cc7784b259c7d403ab5e26da7f5f32c0

Document usage of lowercase, make lowercase optional, add tests (#31) This also documents and throws error when no pool is set to true

view details

dependabot[bot]

commit sha 51d8af68fe21ac6315690fdd2d2c4717b8bd9735

Bump eslint from 2.13.1 to 6.5.1 (#29) Bumps [eslint](https://github.com/eslint/eslint) from 2.13.1 to 6.5.1. - [Release notes](https://github.com/eslint/eslint/releases) - [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md) - [Commits](https://github.com/eslint/eslint/compare/v2.13.1...v6.5.1) Signed-off-by: dependabot[bot] <support@github.com>

view details

Brendan Ashworth

commit sha 009bf378d63077cfd180eeffa42460cc4d9125dd

make lint pass, add npm versions, add package-lock

view details

push time in a month

issue openedtidwall/tile38

Thoughts on sparse field support?

Hi, Thanks for the application, it's really great. For my use case of tile38, I end up with a large number of unique fields set in the same key, where many of those objects don't have the field set (say, 20% of unique fields are popular, whereas 80% of fields are sparse). The current tile38 design for field storage is optimized exclusively for the case where all fields are set. Memory is allocated as though all fields will be filled for every object, so effectively there's an immense memory waste in my use case. Additionally, the default field value of 0 is particularly intuitive for these kinds of queries. It makes querying the set of objects where the field may not be present a pain. I'd like to inquire about whether or not there's interest in supporting the sparse field use case from a performance perspective. I have an implementation of a different field storage that's far more efficient in the sparse use case, but I think there's an inherent trade off between having performance in the sparse case and the saturated case. You must add overhead for sparse fields that would reduce performance in the saturated case. From the benchmarks I've run, performance is order/s of magnitude better for sparse fields, and perhaps ~10% slower for saturated fields. Would a pull request with these changes be supported? Note: the benchmarks I ran were specifically for fields (I think collection.Set with many fields), and if I were to guess, the limiting performance factor is elsewhere on queries.

created time in a month

startedgo-gota/gota

started time in 3 months

pull request commentbrendanashworth/generate-password

Document lowercase restriction, and add lowercase false option.

@donedgardo this looks good, thank you for putting this together. Could you add a new error thrown when the pool.length at the end is zero, before the internal generate is called? Now setting lowercase: false is an easy way to empty the pool and adding that friendly error message would make it easier for people to see where they've gone wrong.

Besides that, this looks great, thanks for sending this in.

donedgardo

comment created time in 3 months

pull request commentnodejs/node

doc: move inactive Collaborators to emeriti

Thank you!

Trott

comment created time in 3 months

issue commentbrendanashworth/generate-password

Is it guaranteed that there will be at least one lowercase character when strict mode is true?

I don't believe this is guaranteed, no. There's a test for something similar, but that passes out of lacking other characters, not for fulfilling a strict lowercase requirement.

There is a lowercase rule, which when combined with strict mode should guarantee the presence of a lowercase character.

That option, though, is neither documented nor unit tested, so that should be fixed.

@donedgardo @collin-pham you should be able to combine lowercase: true with strict: true to get this guarantee for the time being. If you want to send a PR with an improvement, I'll review it, otherwise I'll patch this over the weekend.

collin-pham

comment created time in 3 months

issue commentgolang/geo

Implement Polygon.Intersection

@ctessum I'm not sure myself. The projection onto a plane would conserve intersection points but not lengths.. is it naive to assume the lengths are proportional and thus area is conserved after we project back to the sphere?

ctessum

comment created time in 4 months

issue commentgolang/geo

Implement Polygon.Intersection

I'm interested in contributing a PR for this — are there any thoughts on using Martinez-Rueda for the boolean operation algorithm? Or would it be best to port the C++ version accordingly?

ctessum

comment created time in 4 months

pull request commentgeorust/gpx

Small Parser Improvements

@JDemler thanks for putting this together! I believe the purpose of your test case was to show it no longer errors, but I'd love it if we can also demonstrate via test case that the data is actually pulled into memory. Besides that this looks great and I can bring this in. 👍

JDemler

comment created time in 5 months

Pull request review commentgeorust/gpx

Small Parser Improvements

+<?xml version="1.0" encoding="UTF-8"?>+<gpx xmlns="http://www.topografix.com/GPX/1/1" xmlns:gpsies="https://www.gpsies.com/GPX/1/0" creator="AllTrails https://www.alltrails.com/ - Innrunde" version="1.1" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.topografix.com/GPX/1/1 http://www.topografix.com/GPX/1/1/gpx.xsd https://www.gpsies.com/GPX/1/0 https://www.gpsies.com/gpsies.xsd">+  <metadata>+    <name>Innrunde</name>+    <link href="https://www.gpsies.com/">+      <text>Innrunde on AllTrails</text>+      <type>trackOnWeb</type>+    </link>+    <time>2019-09-11T17:08:31Z</time>+    <extensions>+      <gpsies:property>round trip</gpsies:property>+      <gpsies:trackLengthMeter>6998.28996147978</gpsies:trackLengthMeter>+      <gpsies:totalAscentMeter>91.0</gpsies:totalAscentMeter>+      <gpsies:totalDescentMeter>91.0</gpsies:totalDescentMeter>+      <gpsies:minHeightMeter>297.0</gpsies:minHeightMeter>+      <gpsies:maxHeightMeter>317.0</gpsies:maxHeightMeter>+    </extensions>+  </metadata>+  <trk>+    <name>Innrunde on AllTrails</name>+    <link href="https://www.gpsies.com/map.do">

Could you add a test to make sure this link data is parsed and retrievable from result.tracks[0]?

JDemler

comment created time in 5 months

more