text:
route /auth* mess up if a hash sign is used in url

A PrivateBin url looks like this and always have a # sign in the url
https://privatebin.net/?8132015d33f2dcac#EqRtDrbmAr1zYAagpXz5Muz5bs7rzPd7wVemGXWGKDAT
I have it self-hosted and setup auth login page, but results in:
original url: example.com/?7cc67fc6c545fe8e#Cf3dcZgvYWwqapcvPTZ6Xmyggt1PeiT1GMoiLJFUuLnC
on login page: example.com/auth#Cf3dcZgvYWwqapcvPTZ6Xmyggt1PeiT1GMoiLJFUuLnC
after login: example.com/?7cc67fc6c545fe8e
No idea if regex magic can solve this problem, or its a caddy-auth-portal thing.

@akzkak , did you look into “js redirect”? It is used in the cases of pound sign in url path.

Ah i see. Just found on caddy-auth-jwt front page.  Followed setup guide from here:
https://lucaci32u4.xyz/blog/2021/mainsail-gateway.html
edited in enable js redirect.
jwt {
  primary yes
  enable js redirect

...
but no matter where i put the line it errors on Caddy reload.  "Error during parsing: unsupported directive for enable: js redirect"

@akzkak , please wait until next week. I am going to release new versions of plugins. The reason  you are getting errors is because this feature will be  available only in the new release.

authors: akzkak greenpau

repo: greenpau/caddy-auth-portal


text:
when post an envent about list<> , anyplace that Subscribe List<> will get Event...

I hava to use instanceof to avoid add datas that I don't want....Can you add some judgment
  /*instanceof to avoid some bug...*/
@Subscribe(sticky = true)
public void getMettingMem(List<MettingMem.Mem> mettings) {
	for (Object obj : mettings) {
		if (!(obj instanceof MettingMem.Mem)) {
			return;
		}
	}
	this.mettings = mettings;
	Logger.d("msgSize"+mettings.size());
}

Do not use Generic, generic type is erased at runtime
You should wrap your data
class MeetingsEvent {

      public final List<MettingMem.Mem> mettings;

      MeetingsEvent(List<MettingMem.Mem> mettings) {
              this.mettings = mettings
      }
}

@Subscribe(sticky = true)
public void getMettingMem(MeetingsEvent event) {
	this.mettings = event.mettings;
	Logger.d("msgSize"+this.mettings.size());
}

thank U

source: gharchive/issue

authors: Liberations kukuhyoniatmoko

repo: greenrobot/EventBus

license_type: permissive

license_source: bigquery


text:
Added padding, cursorOnHover, customCSVBody, maxHeight control

Found this component really useful in my application but needed a bit more control over a few components. Specifically I used this table to render oData endpoints so everything was done server-side. Added some style control to table options and a customCSVBody property which allows you to configure a seperate call to retrieve bulk data for CSV rather than simply exporting whatever is on the current page.

hey chris thanks for the PR. I like the custom function for csvbody but the styling options I feel like those can be done via a mui theme provided. Did you try that route?

Hey, thanks for the email. Yeah you're probably  right, I did use muitheme overrides to do them in the first place but the code didn't look as neat haha. Not fussed if you don't want to keep them. The padding option however I think is useful to keep because it is a custom property on the original MuiTable API, it's not simply CSS like the others.
Cheers,
Chris.
On 8 Mar. 2019 8:13 pm, gregn notifications@github.com wrote:
hey chris thanks for the PR. I like the custom function for csvbody but the styling options I feel like those can be done via a mui theme provided. Did you try that route?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgregnb%2Fmui-datatables%2Fpull%2F474%23issuecomment-470908486&data=02|01||e8fa542347734d05f55a08d6a3bf76df|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636876440065655063&sdata=lPteWMt5PBf2dj6TfXe8Hez4G7YhVkAFVVS74wAk8LI%3D&reserved=0, or mute the threadhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FArh9R58KjiG2mHwgW5HdYHTRRbaaKO-Vks5vUlPkgaJpZM4bk1_G&data=02|01||e8fa542347734d05f55a08d6a3bf76df|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636876440065665085&sdata=Jq50kcJremFogt7hkNStc2VkLEsUOmeUWkklmcbOt%2FI%3D&reserved=0.

Hey @chrisswordfish I'm going to decline the PR as I don't want to bloat the API surface. Thank you for sending.

source: gharchive/pull-request

authors: chrisswordfish gregnb

repo: gregnb/mui-datatables

license: MIT

license_source: github-api


text:
Add getting started to readme

README Update
Please add a getting started section to the README so folks know how to start this project

Added Getting Started section to README file.

repo: gregrieck/website


text:
[IDEA] Add Playground prompt examples to gretel blueprints repo.

Problem
Playground prompt examples come to us from product. When we initially built out playground, they changed frequently, and every update required a code change and deploy. This took up engineering time that could have been used for other things. Example prompts change less frequently now, but we still update them occasionally. If we launch new capabilities on Playground, we'll likely have some iteration on them again.
Solution
Move the prompt examples to this repo, where product can more easily adjust them with less engineer involvement (increased efficiency -- a win!). We can also iterate all we want in development environments here, and only move examples to production when we are ready to do so. No feature flag shenanigans required!
Notes
This was an idea Stefan and I have been kicking around for a while. Totally open to adjusting things here if something is easier for product folks to edit. Ideally, I would like to keep these in one file for now for ease of fetching on the Console side.

This would be super helpful for making edits to the prompts!

@thesteady very nice! Does the sequence affect ordering in the UI like with blueprints in gretel.json? And can the id be used for tracking clicks in GTM?

@thesteady very nice! Does the sequence affect ordering in the UI like with blueprints in gretel.json? And can the id be used for tracking clicks in GTM?

@yamini:

Within each grouping, yes, ordering is what is shown in Console.
The id IS used today for GTM clicks 🎉
I have a todo to write up notes about some of this stuff in the Blueprint Management wiki.

authors: hhpang thesteady yamini

repo: gretelai/gretel-blueprints


text:
unable to send EOS tokens from eos-voter wallet

i receive the following error when i attempt to send eos tokens i can vote but not send tokens.

I am having the same issue, its been since yesterday, i cant send my eos out of my eos-voter wallet either, whats going on?

not sure i just downloaded scatter and made do waiting on a response

my scatter wallet wont send either, really starting to be concerned

In the upper left, does it show accountname@active in the dropdown, with both the account name and the permission?
I'm not able to reproduce this issue in 0.7.6 with a fresh wallet. One thing that might fix it is if you reset the application (Tools -> Reset Application), then reimport your keys/accounts to start a new wallet. It's possible if you have an older version, one of the upgrades mangled the wallet somehow.

ok ill try that my friend, thanks!

Closing this issue out - feel free to reopen if the problem persists after reimporting.

authors: Rendiesel23 aaroncox redeyedassassin

repo: greymass/eos-voter


text:
rexolved.

The site ( http://download.gridcoin.us/download/downloadstake/GridcoinResearch.msi ) is giving me install package 41.5 = v3.5.5.9 wallet.
I heard you were looking to serve the packages over https... I think something might have gotten mixed up. It was most definitely v3.5.8.0 before, as I have it installed on my old machine.

i'm an idiot. everything is fine. 🔪

repo: gridcoin/Gridcoin-Research


text:
Segmentation fault when accessing the windowID (Ubuntu)

I have a segmentation fault on Ubuntu 16.04 at this line (it's working on windows 7) :
SDL_Window *win = SDL_GetWindowFromID(window->context->windowID);
Here is the code for the test :
#include <iostream>
#include "gl_core_3_2.h"

#include "SDL.h"
#include "SDL_gpu.h"

int main() {
	GPU_Target* window = GPU_InitRenderer(GPU_RENDERER_OPENGL_3, 800, 600, GPU_DEFAULT_INIT_FLAGS);

	if (window == NULL) {
		GPU_LogError("Initialization Error: Could not create a OpenGL 3.2 renderer.\n");
		return 0;
	}

	SDL_Window *win = SDL_GetWindowFromID(window->context->windowID);
	return 0;
}

I'll take a look at this tomorrow!

I wasn't able to reproduce this yet.  Can you send me your gl_core_3_2.h and gl_core_3_2.c?  Does this sample work for you if you don't include the gl_core_3_2 header?

Same error without gl_core_3_2.h  and  gl_core_3_2.c (generated with glLoadGen).
gl_core_3_2.tar.gz

Problem solved! The error was caused by including another SDL.h file not compatible with the package libsdl2-dev.

Oh, excellent!

license: mit


text:
Is there a way to run this with reuse_port: true ?

Hello I'm working on having grip ready for TechEmpowerBenchmarks
I want to use all the cores available, so I was wondering if there is an option for passing
reuse_port: true

when running the application.
Example of what we do at Kemal
Thanks!

Hi there,
I am very glad for your time and effort, I was too busy to add Grip to the TechEmpowers benchmark and you've gone and done it!
Here is an example where the server is using a reuse port flag.
Thank you!

Perfect, I will update the code.
Thanks!

@grkek congratulations, according to the benchmarks you have created one of the fastest crystal frameworks so far 🙌

authors: CaDs grkek

repo: grip-framework/grip


text:
Duck Duck Go ValueError

[x] I have read and agree to the contributing guidelines.

Describe the bug
We rely on duckduckgo_search for DuckDuckGoWebSearchDriver. It appears that the repo no longer maintained.  We now get this error.
To Reproduce
poetry run python docs/griptape-framework/drivers/src/web_search_drivers_3.py

Expected behavior
No crash.

Deprecating for now, will be removed in 2.0.

repo: griptape-ai/griptape


text:
Added a persistable version of LocalVectorStoreDriver. Added tests fo…

Implemented a persistable child class of LocalVectorStoreDriver that uses pickle to save the internal entries dict.
Added test for the serialization and deserialisation of the new class.

I guess it was just a bad habit. I’ll modify it to use json, and will add an optional argument to save it as zipped json.

I addressed the comments. Not sure why the Quality / lint job complains e.g.
with open(...) as f:
    ....

is pretty idiomatic python . " Variable name "f" doesn't conform to snake_case naming style" might be overkill.
I can change these things i.e. "f->file", but I am not sure it increases readability.

Thanks for adding the changes! Will review tomorrow. Don't worry about squashing—I squash all commits on merge.

Hi @tarbaig we are going to close this PR for now in favor of an implementation that will come through this issue. If you have any interest in working on the other issue, we'd be happy to assist. Thank you!

authors: collindutter tarbaig vasinov


text:
Use feature flags

Tutorial

It is nive to have feature flags dependencies. When feature flag is dependent on another feature flag

Video

repo: gritcsenko/HomeInventory


text:
Token Selection / Hover Clears on Grid Change

When the grid is recalculated, or the selected token moves off-screen then comes back, the rendering does not reset the graphical indication.
Looks like an event needs to run to tell the render loop on update to send a redraw.  There may be other ways, but this seams the easiest and less future-buggy.

Fixed.

repo: groboclown/wind-flower-game


text:
[Master] release/v1.12.0 QA Sign-off

This PR is automatically created by CI to release a new official version of caver-js-ext-kas.
When this PR is approved by QA team, a new version will be released.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: juser0719:x: hackartistsYou have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant gx-circleci

repo: ground-x/caver-js-ext-kas


text:
Nested, Sortable Field Type

Add a new field type that allows for nesting Fields.
Example of a simple sortable list of text fields:
{
  "type": "sortable",
  "key": "list_values",
  "label": "Strings",
  "field": "text"
}

Example of a more complex sortable list of fields:
{
  "type": "sortable",
  "key": "sections",
  "label": "Sections",
  "fields" [
    {
      "type": "text",
      "key": "title",
      "label": "Title"
    },
    {
      "type": "text",
      "key": "subtitle",
      "label": "Sub Title"
    },
  ]
}

We probably want this to work with arbitrary levels of nesting. :) So a
list could embed other lists. Contentful does this by opening a new
tile/lightbox for every sub-list.
On Fri, Feb 14, 2020 at 3:33 PM Randy Merrill notifications@github.com
wrote:

Add a new field type that allows for nesting Fields.
Example of a simple sortable list of text fields:
{
"type": "sortable",
"key": "list_values",
"label": "Strings",
"field": "text"
}
Example of a more complex sortable list of fields:
{
"type": "sortable",
"key": "sections",
"label": "Sections",
"fields" [
{
"type": "text",
"key": "title",
"label": "Title"
},
{
"type": "text",
"key": "subtitle",
"label": "Sub Title"
},
]
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/grow/selective-edit/issues/5?email_source=notifications&email_token=AAE527PCXJRQMSN3VQMO4H3RC4S57A5CNFSM4KVSB3A2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4INWPKEA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE527OTIAPNNXDC6BILTV3RC4S57ANCNFSM4KVSB3AQ
.

It already is setup to work with arbitrary levels, its mostly about how we style it.

authors: Zoramite jeremydw

repo: grow/selective-edit


text:
Running tests with Python3.5 and 3.6 actually runs with 2.7

On my machine (Ubuntu 17.04) with both /usr/bin/python3.5 and /usr/bin/python3.6 available, an invocation of ./tools/run_tests/run_tests.py --language=python --compiler=all_the_cpythons --newline_on_success brings about the creation of py35_native and py36_native directories. That's great.
But then the Pythons in those virtual environments turn out to be symlinks to python2: py35_native/bin/python --version prints "Python 2.7.13".
If there's some problem constructing the py35 and py36 environments, then construction should fail rather than silently fall back to Python 2.

That's something we do in build_python.sh . This was added 2 years ago in #6791, and I'm okay with removing this because it leads to misleading results.
Just making sure though, after looking at the context from that lengthy PR, do you still prefer to get rid of the fallback?

@matt-kwong: after reviewing the context... yes, I think it's the right call to fail if the appropriate Python interpreter isn't available rather than use another Python interpreter.

authors: matt-kwong nathanielmanistaatgoogle

repo: grpc/grpc


text:
Any plan to support opencensus in grpc c core?

Is your feature request related to a problem? Please describe.
There is a census.h in c library that allow user to set a census_context to a call. But there is no way to construct a census_context. And as far as I can see, opencensus are mainly maintained in the cpp land. Do you have any plan to support opencensus inside the grpc c core?
Describe the solution you'd like
Support opencensus in grpc c core and provide a way to contruct census context using C API.
Describe alternatives you've considered
No.
Additional context
Nothing.

I don't think this question is ever answered or even responded. Friendly ping @donnadionne.

We do support opencensus in C-core. But it only works when building with bazel, not with cmake.

For the future, please direct general question like this one to  https://stackoverflow.com/search?tab=newest&q=grpc is%3Aquestion
Your answer may already be there or you can post a new question if it's not.

authors: BusyJay donnadionne


text:
Bazel C/C++ Opt MacOS: //test/cpp/end2end:client_callback_end2end_test failed due to SIGSEGV

https://source.cloud.google.com/results/invocations/6804105c-50dc-418f-bb90-873eb8d7d48b/log

Caught signal SIGSEGV
no backtrace
external/bazel_tools/tools/test/test-setup.sh: line 310: 98643 Segmentation fault: 11  "${TEST_PATH}" "$@" 2>&1
[10,546 / 10,554] 1656 / 2536 tests, 1 failed; Testing //test/core/end2end:h2_census_test@high_initial_seqno; 20s remote-cache, darwin-sandbox ... (16 actions running)
INFO: From Compiling test/core/channel/channel_args_test.cc:
test/core/channel/channel_args_test.cc:141:13: warning: unused function 'test_channel_create_with_global_mutator' [-Wunused-function]
static void test_channel_create_with_global_mutator(void) {
^
1 warning generated.
[12,094 / 12,111] 2129 / 2536 tests, 1 failed; Testing //test/core/end2end:h2_http_proxy_test@cancel_after_invoke; 35s remote-cache, darwin-sandbox ... (16 actions running)
[12,893 / 12,901] 2380 / 2536 tests, 1 failed; Testing //test/core/end2end:h2_tls_test@cancel_after_invoke; 11s remote-cache, darwin-sandbox ... (16 actions running)
INFO: Elapsed time: 1861.292s, Critical Path: 270.17s
INFO: 8049 processes: 8049 darwin-sandbox.
INFO: Build completed, 1 test FAILED, 13354 total actions

Closing since we decided not to track test flakes on github

authors: karthikravis yashykt


text:
Build fails

Build is failing on fresh clone in Ubuntu
[HOSTCXX] Compiling src/compiler/csharp_generator.cc
In file included from ./src/compiler/config.h:37:0,
                 from ./src/compiler/csharp_generator.h:37,
                 from src/compiler/csharp_generator.cc:39:
include/grpc++/impl/codegen/config_protobuf.h:38:42: fatal error: google/protobuf/stubs/common.h: No such file or directory
 #include <google/protobuf/stubs/common.h>

Did you git submodule update --init?
If no, please try it. If yes, what are the commands you used starting from the clone and ending in the failure?

Issued the standard commands mentioned in instructions. Following is the sequence of commands.
$ git clone -b $(curl -L http://grpc.io/release) https://github.com/grpc/grpc
$ cd grpc
$ git submodule update --init
$ make

The make fails with protobuf related error.

You should make protobuf manual firstly.

It was working two weeks back. Now it doesn't. The following paragraph in INSTALL.md does not make sense. It is completely misleading.

##Protoc
By default gRPC uses protocol buffers,
you will need the protoc compiler to generate stub server and client code.
If you compile gRPC from source, as described below, the Makefile will
automatically try and compile the protoc in third_party if you cloned the
repository recursively and it detects that you don't already have it
installed.

When you run git submodule update --init after that it will update protobuffer, so you need re-compile protobuffer. Maybe you should try it. Thanks.

I have successfully built gRPC and brotobuf individually (after downloading sources separately). But the option of building both from source using gRPC's Makefile does not seem to work (for me).
As per thinkerou's suggestion, when I try to re-compile protobuf in third_party folder it fails with gmock error
$ ./autogen.sh
Google Mock not present.  Fetching gmock-1.7.0 from the web...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1582  100  1582    0     0   3481      0 --:--:-- --:--:-- --:--:--  3484
[gmock-1.7.0.zip]
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of gmock-1.7.0.zip or
        gmock-1.7.0.zip.zip, and cannot find gmock-1.7.0.zip.ZIP, period.

Probably this error message is not related to gRPC development team.

It have the contents about gmock in autogen.sh
https://github.com/google/protobuf/blob/bba446bbf2ac7b0b9923d4eb07d5acd0665a8cf0/autogen.sh#L34

You shouldn't need to re-compile protobuf in the third_party folder.
Do you have protobuf installed elsewhere on your system? Specifically a version lower than that pulled by gRPC's reference to the protobuf repository (3.0.x)?

The latest clone seems to be building fine.

authors: damleak soltanmm-google thinkerou


text:
Add escape sequences for the grpc_cli metadata flag

Add escape character '\', so that colons can be included in the metadata values in grpc_cli.
In the metadata flag:

Sequence "\:"  represents character ':'
Sequence "\\" represents character '\'

Fixes: #10951

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

It's been a while since last time I used this gif

authors: grpc-testing y-zeng


text:
Change xds plugin name to xds_experimental until it's ready for use.

This ensures that anyone using an older grpc build will not be affected when the resolver starts telling clients to use the xds policy.

****************************************************************

libgrpc.so

     VM SIZE              FILE SIZE
 ++++++++++++++ GROWIN ++++++++++++++
  +0.0%     +16 [None]       0  [ = ]

  +0.0%     +16 TOTAL        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 2,019,735      Total (>)      2,019,718

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
11,154,116      Total (=)     11,154,116

 No significant differences in binary sizes

Corrupt JSON data (indicates timeout or crash):
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.new: 10
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.old: 10

[microbenchmarks] No significant performance differences

authors: grpc-testing markdroth


text:
[Security - Revocation] Refactor how CRLs are checked internally

This PR changes how CRLs are handled purely internally. After discussing with davidben@, there are various problems with the X509_STORE_set_get_crl API and we shouldn't use it. This change keeps the behavior and external API the same, but instead of bulk pushing CRL information into OpenSSL, we instead iterate through the built chain and check each certificate for revocation, as well as doing the CRL validation ourselves.

Thanks for the PR Greg! Don't we need tests that guard the new functionality introduced in this PR?

Friendly ping on the testing question. :)

Ah missed that, sorry! The existing tests should cover all of this as it's purely internal logic. If implemented correctly no behavior should change

Ah missed that, sorry! The existing tests should cover all of this as it's purely internal logic. If implemented correctly no behavior should change

Do we at least want tests that ensure we're correctly checking the signature on the CRL? Isn't that a functional difference between the current state and this PR?

Added to the tests, PTAL


text:
[skip ci] Update CODEOWNERS

We recently adopted a new team structure in. One of the things we need to do is update the CODEOWNERS file for each of the repos we own. Since I want to reduce noise, I am taking the approach where I will be the sole owner for the repos I lead (so that I get notified and assigned), but have others in the team in the file as a comment so that I know who I can solicit for help should it get overwhelming.

Going to merge this now.

repo: gruntwork-io/terraform-kubernetes-helm


text:
--terragrunt-include-dir includes only one level of dependencies

On Terragrunt v0.36.0.
A command like tg run-all apply --terragrunt-include-dir A will include the direct dependencies of A, but it does not reliably include the dependencies of those dependencies.
I think part of the problem might be here: https://github.com/gruntwork-io/terragrunt/blob/v0.36.0/configstack/module.go#L203
This code includes the dependencies of directories that match --terragrunt-include-dir globs, but does not then recurse into those dependencies to include their dependencies.
I say it doesn't "reliably" include them because, in my stack, I have one dependency being included that isn't a direct dependency of the --terragrunt-include-dir module, and I can't figure out why. (Possibly related to something like #808, although I tried renaming folders to change their alphabetical ordering, and couldn't get the behavior to change.)
An anonymized version of my directory layout and dependency graph (which might not be very helpful):
A/
    1/
    2/
    3/
    4/
    5/
B/
    1/
    2/
    3/
C/
    1/

digraph {
        "A/1" ;
        "A/2" ;
        "A/2" -> "A/1";
        "A/2" -> "A/3";
        "A/2" -> "A/5";
        "A/3" ;
        "A/3" -> "A/5";
        "A/4" ;
        "A/4" -> "B/1";
        "A/4" -> "A/2";
        "A/5" ;
        "B/1" ;
        "B/1" -> "A/2";
        "B/2" ;
        "B/2" -> "A/2";
        "B/3" ;
        "B/3" -> "A/2";
        "B/3" -> "A/4";
        "C/1" ;
        "C/1" -> "B/2";
        "C/1" -> "B/3";
        "C/1" -> "A/2";
}

The command I'm running, from the root (where A, B, and C are), is:
terragrunt run-all apply --terragrunt-ignore-external-dependencies --terragrunt-include-dir C/1

I expect this command to include everything in this graph, since they're all dependencies, but instead, I only get:
Group 1
- Module A/2

Group 2
- Module A/4
- Module B/2

Group 3
- Module B/3

Group 4
- Module C/1

Notice that all direct dependencies of C/1 are included. But so is A/4, for some reason. Everything else is for some reason excluded.

We're also experiencing this (version 0.36.2), tested with the following structure:
/foo
  /main.tf
  /terragrunt.hcl
/bar
  /main.tf
  /terragrunt.hcl
    dependency "foo" {
      config_path = "../foo"
    }
/baz/a
  /main.tf
  /terragrunt.hcl
    dependency "bar" {
      config_path = "../../bar"
    }

Running terragrunt run-all apply --terragrunt-include-dir 'baz/*'
Group 1
- Module /home/ondrej/www/yr/playground/terragrunt/bar

Group 2
- Module /home/ondrej/www/yr/playground/terragrunt/baz/a

only projects to apply modules bar and baz/a, when all 3 should be. The problem persists even when the targeted module is not nested.
One possible workaround for the nested case might be running terragrunt run-all --terragrunt-include-external-dependencies apply inside the /3/a subdirectory. This does correctly project to apply all 3 modules.

authors: geekofalltrades ondrejsson

repo: gruntwork-io/terragrunt


text:
Use OpenTofu by default if available in environment

Description
Use OpenTofu by default if available in environment

TODOs
Read the Gruntwork contribution guidelines.

[x] Update the docs.
[ ] Run the relevant tests successfully, including pre-commit checks.
[x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[x] Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Added / Removed / Updated [X].
Updated Terragrunt to check by default for OpenTofu executable before switching to Terraform.
Migration Guide

I approved, but it seems like the integration test for Terraform needs to be updated so that TERRAGRUNT_TFPATH is explicitly set to terraform.


text:
http_helper.HttpGetWithRetry does not use global proxy settings

A proxy has been configured on my machine (env http_proxy etc), about the test run, the method cannot reach my ec2 machine and the http.Get method can. Judging by the source, it looks like the library uses its own http transport and does not use the global proxy settings, as Go himself does. Correct me if I'm wrong

Here's how we initialize http.Transport: https://github.com/gruntwork-io/terratest/blob/master/modules/http-helper/http_helper.go#L35-L37. Would you be up for a PR to use the defaults for from DefaultTransport?

Do let me make it's clear: I have a Terraform module that deploys an EC2 machine to AWS and a Go test that sends a request to this machine (code is below). The problem is the EC2 machine is available to me only through the proxy that I configured.
My code:
url: = terraform.Output (t, terraformOptions, "url")
status: = 200
text: = "Hello, World"
retries: = 5
sleep: = 40 * time.Second
http_helper.HttpGetWithRetry (t, url, nil, status, text, retries, sleep)

Steps to reproduce:
You need a remote machine that is not accessible without a proxy
Current behavior:
[...]
TestWebServer 2020-06-10T16:42:39+04:00 retry.go:72: HTTP GET to URL http://54.162.238.165:8080
TestWebServer 2020-06-10T16:42:39+04:00 http_helper.go:32: Making an HTTP GET call to URL http://54.162.238.165:8080
TestWebServer 2020-06-10T16:42:49+04:00 retry.go:84: HTTP GET to URL http://54.162.238.165:8080 returned an error: Get "http://54.162.238.165:8080": context deadline exceeded (Client.Timeout exceeded while awaiting headers). Sleeping for 40s and will try again.
TestWebServer 2020-06-10T16:43:29+04:00 retry.go:72: HTTP GET to URL http://54.162.238.165:8080
TestWebServer 2020-06-10T16:43:29+04:00 http_helper.go:32: Making an HTTP GET call to URL http://54.162.238.165:8080
TestWebServer 2020-06-10T16:43:39+04:00 retry.go:84: HTTP GET to URL http://54.162.238.165:8080 returned an error: Get "http://54.162.238.165:8080": context deadline exceeded (Client.Timeout exceeded while awaiting headers). Sleeping for 40s and will try again.
[...]

Expected behavior:
[...]
Making an HTTP GET call to URL http:///54.162.238.165:8080
Success!
[...]

Yup, I understand! I think the only way to support that for now with Terratest helpers is to modify how we configure http.Transport to leverage DefaultTransport: https://github.com/gruntwork-io/terratest/blob/master/modules/http-helper/http_helper.go#L35-L37. If you're up for a PR to do that, we'd welcome it!

@miranius @brikis98  I believe the cause of issue is that when initializing http.Transport, the Proxy field value was not set with the configured proxy http_proxy on your machine.
I have a raised a pull request 560 to address this issue.

authors: alexkayabula brikis98 miranius

repo: gruntwork-io/terratest


text:
🛑 kinderundjugendtanz.at is down

In 23858b6, kinderundjugendtanz.at (https://kinderundjugendtanz.at) was down:

HTTP code: 0
Response time: 0 ms

Resolved: kinderundjugendtanz.at is back up in 980b174.


text:
🛑 volkstanz.st is down

In 3812c30, volkstanz.st (https://volkstanz.st) was down:

HTTP code: 0
Response time: 0 ms

Resolved: volkstanz.st is back up in 4492928.


text:
🛑 kinderundjugendtanz.at is down

In 0b84d23, kinderundjugendtanz.at (https://kinderundjugendtanz.at) was down:

HTTP code: 0
Response time: 0 ms

Resolved: kinderundjugendtanz.at is back up in 797c36f.


text:
🛑 volkstanz.st is down

In e87e7ed, volkstanz.st (https://volkstanz.st) was down:

HTTP code: 0
Response time: 0 ms

Resolved: volkstanz.st is back up in a24eb8c.


text:
🛑 bretterhofer.at is down

In 919ef98, bretterhofer.at (https://bretterhofer.at) was down:

HTTP code: 0
Response time: 0 ms

Resolved: bretterhofer.at is back up in ba65a71.


text:
🛑 volkstanzwannwo.at is down

In f764ee6, volkstanzwannwo.at (https://volkstanzwannwo.at) was down:

HTTP code: 0
Response time: 0 ms

Resolved: volkstanzwannwo.at is back up in dfddb82.


text:
🛑 volkstanzwannwo.at is down

In 7743e20, volkstanzwannwo.at (https://volkstanzwannwo.at) was down:

HTTP code: 0
Response time: 0 ms

Resolved: volkstanzwannwo.at is back up in 9da9e17.


text:
🛑 h4.bretterhofer.at is down

In d0ba65b, h4.bretterhofer.at (https://h4.bretterhofer.at/toggle.php) was down:

HTTP code: 0
Response time: 0 ms

Resolved: h4.bretterhofer.at is back up in 4a76193.


text:
🛑 volkstanzwannwo.at is down

In ba2f41b, volkstanzwannwo.at (https://volkstanzwannwo.at) was down:

HTTP code: 0
Response time: 0 ms

Resolved: volkstanzwannwo.at is back up in 98ca2b0.


text:
🛑 volkstanz.st is down

In b8d7f44, volkstanz.st (https://volkstanz.st) was down:

HTTP code: 0
Response time: 0 ms

Resolved: volkstanz.st is back up in 81a2a40 after 16 minutes.